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
Add module-expression #238
Conversation
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
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 with a nit.
type: "ModuleExpression"; | ||
body: Program; | ||
} | ||
``` |
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.
Consider add a note that the sourceType
of body
must be "module"
.
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 can probably subtype Program
to a Module
for that. But it's fine for experimental.
babel/babel#12469 has been merged and Babel 7.13 has been released! Can we merge this? |
ping @RReverser |
@nzakas Don't you think this will break losts of ESLint plugins? Many rule listen on |
Yes, that’s a good point, which is why we probably want to go with @RReverser’s suggestion in the final version: #238 (comment) |
View Rendered Text
Adds spec for JS Module Blocks. (ref: babel/babel#12469 (comment))