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

[Types] fix generated TS/Flow comment types #9007

Merged
merged 1 commit into from Nov 9, 2018

Conversation

ljqx
Copy link
Contributor

@ljqx ljqx commented Nov 9, 2018

Q                       A
Fixed Issues? Fixes #8969
Patch: Bug Fix? Yes
Major: Breaking Change? No
Minor: New Feature? No
Tests Added + Pass? No need
Documentation PR Link None
Any Dependency Changes? No
License MIT

As described in #8969, the generated TS/Flow types for comments are not aligned with the ones used by @babel/parser. This change makes them same as the real types used in code.
Specifically:
BlockComment => CommentBlock
LineComment => CommentLine

@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/9367/

@xtuc xtuc added the pkg: types label Nov 9, 2018
@existentialism existentialism added the PR: Bug Fix 🐛 A type of pull request used for our changelog categories label Nov 9, 2018
@existentialism existentialism merged commit 4dfd801 into babel:master Nov 9, 2018
@existentialism
Copy link
Member

@ljqx thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: types PR: Bug Fix 🐛 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@babel/types BaseComment type not match @babel/parser output
5 participants