Skip to content

[parser] Parse only modifiers of actual methods#10594

Merged
nicolo-ribaudo merged 2 commits intobabel:masterfrom
zant:bugfix/optional-async-error
Oct 29, 2019
Merged

[parser] Parse only modifiers of actual methods#10594
nicolo-ribaudo merged 2 commits intobabel:masterfrom
zant:bugfix/optional-async-error

Conversation

@zant
Copy link
Copy Markdown
Contributor

@zant zant commented Oct 23, 2019

Q                       A
Fixed Issues? Follow-up from #10582 comment
Patch: Bug Fix? Yes
Major: Breaking Change? No
Minor: New Feature? No
Tests Added + Pass? Yes
Any Dependency Changes? No
License MIT

i've did what in my opinion was the most trivial solution and added a test as an start point, reviews are highly appreciated

@nicolo-ribaudo nicolo-ribaudo added area: typescript PR: Bug Fix 🐛 A type of pull request used for our changelog categories labels Oct 25, 2019
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung left a comment

Choose a reason for hiding this comment

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

Nice work!
The CI error is unrelated.

@JLHwung
Copy link
Copy Markdown
Contributor

JLHwung commented Oct 29, 2019

@gonzarodriguezt Could you rebase on master?

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.

Thanks!

@nicolo-ribaudo nicolo-ribaudo merged commit b6ef968 into babel:master Oct 29, 2019
@zant
Copy link
Copy Markdown
Contributor Author

zant commented Oct 29, 2019

@JLHwung Sorry i was afk!

It's a pleasure @nicolo-ribaudo 🙂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: typescript outdated A closed issue/PR that is archived due to age. Recommended to make a new issue 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.

3 participants