-
-
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 README for Babel TypeScript Plugin #7443
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/6975/ |
## Workarounds | ||
|
||
* `namespace`: Migrate to defining modules instead. | ||
* `const enum`: Avoid to using enums outside of current file |
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.
You can just change to enum
(without const
), which is available as a value at runtime and can be transpiled without type information.
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.
Thanks @andy-ms - updated 👍
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.
I prefer "compile" over "transpile" but I don't care enough to request changes. 😄 LGTM
@orta thanks! |
Provides additional docs on what the trade-offs are for moving to Babel 7, and a few work-arounds.
The one that I'm not 100% on is the
const enum
s. Would love some help on what the recommendation for replacing that could look like?/cc @andy-ms @existentialism @DanielRosenwasser for ideas on that