Skip to content
This repository was archived by the owner on Jan 19, 2019. It is now read-only.

Add support async iterators #239

Merged
merged 1 commit into from
May 1, 2017
Merged

Conversation

soda0289
Copy link
Member

This commit depends upon #237. Since that PR adds tests for async generators and allows experimental ecma-features tests to run.

https://github.com/estree/estree/blob/master/experimental/async-iteration.md

@soda0289 soda0289 requested a review from JamesHenry April 29, 2017 19:13
@eslintbot
Copy link

LGTM

@JamesHenry
Copy link
Member

Requisite PR merged, just let me know whenever this is ready to be reviewed

@soda0289 soda0289 force-pushed the add-support-async-iterators branch from d30cc2f to 401cd66 Compare April 30, 2017 16:37
@eslintbot
Copy link

Thanks for the pull request, @soda0289! I took a look to make sure it's ready for merging and found some changes are needed:

  • The commit summary must be 72 characters or shorter. Please check out our guide for how to properly format your commit summary and update it on this pull request.

Can you please update the pull request to address these?

(More information can be found in our pull request guide.)

@soda0289 soda0289 force-pushed the add-support-async-iterators branch from 401cd66 to 69a0ffa Compare April 30, 2017 16:38
@eslintbot
Copy link

LGTM

@soda0289
Copy link
Member Author

@JamesHenry should be good to review

Copy link
Member

@JamesHenry JamesHenry left a comment

Choose a reason for hiding this comment

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

Thanks!

@JamesHenry JamesHenry merged commit e311620 into master May 1, 2017
@JamesHenry JamesHenry deleted the add-support-async-iterators branch May 1, 2017 10:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants