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

Fix code generation for async generator methods #6998

Merged
merged 2 commits into from
Dec 8, 2017

Conversation

Jessidhia
Copy link
Member

@Jessidhia Jessidhia commented Dec 8, 2017

Q                       A
Fixed Issues? 🙆‍♀️
Patch: Bug Fix? 🤘
Major: Breaking Change? 🙅‍♀️
Minor: New Feature? 🤷‍♀️
Tests Added + Pass? 👍
Any Dependency Changes? 👎
License MIT

babel-generator is writing async generator methods as *async methodName() {} instead of async *methodName() {}. This puts the * in the right place.

@Jessidhia Jessidhia added pkg: generator PR: Bug Fix 🐛 A type of pull request used for our changelog categories Spec: Async Generators labels Dec 8, 2017
@babel-bot
Copy link
Collaborator

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

1 similar comment
@babel-bot
Copy link
Collaborator

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

@Jessidhia Jessidhia merged commit 2958548 into babel:master Dec 8, 2017
@Jessidhia Jessidhia deleted the fix-async-generator-generation branch December 8, 2017 08:38
larixer added a commit to larixer/babel that referenced this pull request Jul 25, 2018
hzoo pushed a commit that referenced this pull request Jul 25, 2018
babel-generator is writing async generator methods as `*async methodName() {}` instead of `async *methodName() {}`. This puts the `*` in the right place.

This is a port of PR: #6998 for `babel@6.x`
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 5, 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 pkg: generator PR: Bug Fix 🐛 A type of pull request used for our changelog categories Spec: Async Generators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants