Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: removing babel #4560

Merged
merged 1 commit into from
Nov 28, 2023
Merged

test: removing babel #4560

merged 1 commit into from
Nov 28, 2023

Conversation

kyranjamie
Copy link
Collaborator

@kyranjamie kyranjamie commented Nov 22, 2023

Try out this version of Leather — download extension builds.

Anyone please feel free to have a play around with these settings as well.

I was under the impression removing babel would drastically speed up the build. It doesn't, though still removes a handful of packages.

Bonus points if anyone can get yarn build sub 5s and the dev build to rebuild within 1s.

@kyranjamie kyranjamie force-pushed the experimental/remove-babel branch from 343d17a to 0425d24 Compare November 22, 2023 14:01
@kyranjamie kyranjamie force-pushed the experimental/remove-babel branch from 0425d24 to e51e710 Compare November 22, 2023 14:08
@pete-watters
Copy link
Contributor

How about just merging this to get rid of babel now and then we can try and further improve on it later?

@pete-watters pete-watters self-requested a review November 28, 2023 06:00
@kyranjamie
Copy link
Collaborator Author

@pete-watters can you checkout & confirm sanity check it's working correctly? Then happy to merge

@kyranjamie kyranjamie marked this pull request as ready for review November 28, 2023 10:50
@pete-watters
Copy link
Contributor

@pete-watters can you checkout & confirm sanity check it's working correctly? Then happy to merge

I tested it last week, trying and failing to get the bonus points.

I retested now and LGTM. I can build the APP and things look the same

@kyranjamie kyranjamie added this pull request to the merge queue Nov 28, 2023
Merged via the queue into dev with commit e284fae Nov 28, 2023
26 checks passed
@kyranjamie kyranjamie deleted the experimental/remove-babel branch November 28, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants