Skip to content

updated ParsePlugin type definition#10259

Merged
nicolo-ribaudo merged 2 commits intobabel:masterfrom
letladi:issue-10186
Jul 26, 2019
Merged

updated ParsePlugin type definition#10259
nicolo-ribaudo merged 2 commits intobabel:masterfrom
letladi:issue-10186

Conversation

@letladi
Copy link
Copy Markdown
Contributor

@letladi letladi commented Jul 23, 2019

Fixes #10186

@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Jul 23, 2019

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

@babel-bot
Copy link
Copy Markdown
Collaborator

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

@JLHwung
Copy link
Copy Markdown
Contributor

JLHwung commented Jul 24, 2019

Could you link the fixed issue in the description? i.e.

Fixes #10186 

It would update the issue status once this PR gets merged.

Protip: When you are creating a pull request, there will be a PR template. Try to spend some time filling in the form, it will help reviewers know what it the type of PR (bugfix/feature/internal) in first glance.

@letladi
Copy link
Copy Markdown
Contributor Author

letladi commented Jul 25, 2019

Thanks for the tip. I will keep that in mind going forward.

Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I now that they were already out of order, but could you sort them alphabetically?

@letladi
Copy link
Copy Markdown
Contributor Author

letladi commented Jul 25, 2019

Ok cool. Will do 👍🏽

@JLHwung JLHwung mentioned this pull request Jul 25, 2019
2 tasks
@nicolo-ribaudo nicolo-ribaudo merged commit f160522 into babel:master Jul 26, 2019
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 25, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2019
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

babel/packages/babel-parser/typings/babel-parser.d.ts lacks some plugins

5 participants