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

[BREAKING CLEANUP beta] Update tsconfig to target ES2021 #20439

Merged
merged 3 commits into from
Apr 18, 2023
Merged

Conversation

chriskrycho
Copy link
Contributor

All of our target browsers and Node versions for Ember v5.0 support ES2021, so update our TS version accordingly.

All of our target browsers and Node versions for Ember v5.0 support
ES2021, so update our TS version accordingly.
Notes:

1. We cannot currently upgrade beyond `broccoli-rollup` v3. Most likely
   this is related to the changes in Rollup v2, which seem to handle
   the circular dependencies more accurately; the net is that we end up
   dropping modules from the compiled output, though, which is
   undesirable (to say the least!).

2. We removed our usage of these rollup plugins some time ago, but we
   failed to drop them from `package.json`.
Since `broccoli-typescript-compiler` v8, we get `typescript` directly
ourselves, and `broccoli-typescript-compiler` uses it as a peer dep
rather than supplying it itself. Remove our custom lookup for it.
@chriskrycho chriskrycho merged commit b1ebc7d into main Apr 18, 2023
@chriskrycho chriskrycho deleted the ts-targets branch April 18, 2023 23:46
@chriskrycho chriskrycho changed the title [CLEANUP BETA] Update tsconfig to target ES2021 [BREAKING CLEANUP beta] Update tsconfig to target ES2021 Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant