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

Addressed syntax deprecation from @babel/plugin-proposal-decorators #36

Merged
merged 1 commit into from
May 31, 2023

Conversation

ijlee2
Copy link
Owner

@ijlee2 ijlee2 commented May 31, 2023

Description

According to Babel, the syntax { "legacy": true } has been deprecated.

Use version: "legacy" instead. This option is a legacy alias.

I upstreamed the change in embroider-build/addon-blueprint#127.

@ijlee2 ijlee2 added the enhance: code Issue asks for new feature or refactor label May 31, 2023
@ijlee2 ijlee2 marked this pull request as ready for review May 31, 2023 04:58
@@ -3,7 +3,7 @@
<% } %> "plugins": [
"@embroider/addon-dev/template-colocation-plugin",<% if (options.packages.addon.hasTypeScript) { %>
["@babel/plugin-transform-typescript", { "allowDeclareFields": true }],<% } %>
["@babel/plugin-proposal-decorators", { "legacy": true }],
["@babel/plugin-proposal-decorators", { "version": "legacy" }],
Copy link
Owner Author

@ijlee2 ijlee2 May 31, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested { "version": "legacy" } with ember-container-query as well as 4 private addons. The tests for these addons passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhance: code Issue asks for new feature or refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant