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

[QUEST] v8 Release #453

Closed
10 tasks done
bertdeblock opened this issue Sep 16, 2022 · 15 comments
Closed
10 tasks done

[QUEST] v8 Release #453

bertdeblock opened this issue Sep 16, 2022 · 15 comments
Assignees

Comments

@bertdeblock
Copy link
Member

bertdeblock commented Sep 16, 2022

@kategengler
Copy link
Member

from @rwjblue in Discord

rwjblue — 09/16/2022
Agree
There’s a non trivial amount of cleanup we should do though
Like bumping the broccoli-Babel-transpiler version
Changing poly fill support around (removing it basically)

@ef4
Copy link
Collaborator

ef4 commented Sep 28, 2022

Also just noticed that broccoli-babel-transpiler has @babel/core in dependencies. That means we should also change and release broccoli-babel-transpiler to either use a peerDep or accept babel itself as an argument when you create the broccoli node.

@ef4
Copy link
Collaborator

ef4 commented Sep 28, 2022

(I can't edit the description above, so I'm rewriting it here with updates.)

  • move @babel/core to peerDependencies Move @babel/core to peerDependencies to resolve peer dependency warnings and errors #452
  • drop support for node < 14 and add test coverage for 14 through 18
  • remove includePolyfill support
  • change broccoli-babel-transpiler to either make @babel/core a peerDep or make it accept the current Babel as an argument, and upgrade ember-cli-babel to that version. (Without this we're still not actually transpiling people's code with the version of @babel/core in their own package.json.)

@bertdeblock
Copy link
Member Author

@ef4 For the broccoli-babel-transpiler update, which option do you prefer / is best in this case?

@bertdeblock
Copy link
Member Author

Also, does includePolyfill support need to be dropped without a replacement?

@bertdeblock
Copy link
Member Author

bertdeblock commented Oct 1, 2022

Created PRs for the remaining issues. Hope I understood everything correctly 😄.

Probably need some guidance on the "Remove includePolyfill support" issue.

@ef4
Copy link
Collaborator

ef4 commented Oct 2, 2022

Also, does includePolyfill support need to be dropped without a replacement?

babel has deprecated the polyfill and tells you the replacement is to add import "core-js/stable" yourself: https://babeljs.io/docs/en/babel-polyfill

This probably belongs in changelog and in any how-to-upgrade doc that we may write for this major release.

@rwjblue
Copy link
Member

rwjblue commented Oct 7, 2022

I released https://github.com/babel/broccoli-babel-transpiler/releases/tag/v8.0.0-beta.1 so we can start updating here a bit easier

@bertdeblock
Copy link
Member Author

👍 I'll try to fix the broccoli-babel-transpiler update here asap.

@ef4
Copy link
Collaborator

ef4 commented Oct 13, 2022

We're going to need to document what addon authors are supposed to do. They traditionally must have ember-cli-babel in dependencies and therefore they bring their own babel into app builds. The way to reproduce that after this change is for addons to @babel/core in dependencies. But it's probably better for them to put it into peerDependencies (and devDependencies).

@bertdeblock
Copy link
Member Author

After the transpiler PR, I can look into writing an upgrade guide for apps and addons.

@ef4
Copy link
Collaborator

ef4 commented Nov 2, 2022

I said about "it's probably better for them to put it into peerDependencies", but after thinking about it, the less disruptive change is to put @babel/core into dependencies. That is equivalent to how things already worked, with every addon potentially bringing its own babel. We don't need to rock that boat any harder, the longer-term plan is for them to get to v2 addons and then they don't bring ember-cli-babel at all.

@JackHowa
Copy link

Excited to see babel polyfills removed 👍

@bertdeblock
Copy link
Member Author

v8 is released.

@ef4
Copy link
Collaborator

ef4 commented Aug 18, 2023

Thank you @bertdeblock!

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 a pull request may close this issue.

5 participants