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
Remove Babylon plugins for features already merged to the ECMAScript spec #8448
Remove Babylon plugins for features already merged to the ECMAScript spec #8448
Conversation
"objectRestSpread", | ||
"classProperties", | ||
); | ||
parserOpts.plugins.push("typescript", "classProperties"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a breaking change if someone uses an older Babylon version. Since we are in RC, I can revert it if you prefer.
74e2fe8
to
a734266
Compare
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/9332/ |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/8795/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, I like keeping stuff in place to avoid the tiny risk of breaking stuff.
d1425ef
to
d80b02a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 w/o breaking change
…spec - objectRestSpread (2018) - asyncGenerators (2018) - optionalCatchBInding (2019) - jsonStrings (2019)
cbd9e01
to
386dea9
Compare
Follows babel/babel#8448 Context in review comment: https://github.com/sindresorhus/refined-github/pull/1449/files#r216110819
These are now enabled by default:
TODO (after this PR):
@babel/plugin-syntax-*
packages.@babel/plugin-proposal-*
packages.@babel/plugin-transform-*
packages.