-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Update syntax-decorators options #7938
Update syntax-decorators options #7938
Conversation
nicolo-ribaudo
commented
May 15, 2018
•
edited
Loading
edited
- Add decoratorsBeforeExport to the syntax plugin
- Require legacy: true, like in the transform plugin
Q | A |
---|---|
Fixed Issues? | |
Patch: Bug Fix? | |
Major: Breaking Change? | |
Minor: New Feature? | |
Tests Added + Pass? | Yes |
Documentation PR | |
Any Dependency Changes? | |
License | MIT |
- Tests (I will enable them in Add support for the new decorators proposal #7976, since currently legacy must be true)
|
||
### `decoratorsBeforeExport` | ||
|
||
`boolean`, defaults to `false`. |
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.
We might need to change the default value after the tc39 meeting
c445aff
to
e3cdb7c
Compare
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/8245/ |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/7976/ |
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.
Not uber important, but it might be useful to denote why the exportBefore option exists.
This option allows developers to experimet both the possible syntaxes. This will | ||
allow collecting an informed feedback from the community, which will help to | ||
decide which the final syntax should be. | ||
[tc39/proposal-decorators#69](https://github.com/tc39/proposal-decorators/issues/69) |
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.
@littledan Is this description ok?
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.
Maybe:
This option was added to help tc39 collect feedback from the community by allowing experimentation with both possible syntaxes.
For more information, check out: tc39/proposal-decorators#69
da8e262
to
d181894
Compare
* Add decoratorsBeforeExport to the syntax plugin * Require legacy: true, like in the transform plugin
d181894
to
638d365
Compare