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.

Sign up for GitHub

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

Jump to bottom

nextLast() instead of next("last") #12

Merged

hax merged 4 commits into main from api-nextLast

Jul 7, 2022

Member

hax commented Jul 3, 2022

fix #11

hax and others added 4 commits

July 3, 2022 19:28


          nextLast() instead of next("last")

466b957

fix #11


          fixup: [spec] npm run build

1aa9f88


          miss next("last")

7ccc375


          another miss

4bba704

Member Author

hax commented Jul 7, 2022

As tc39/proposal-iterator-helpers#122 (comment) , iterator helpers do not pass the protocol, so we have to use nextLast().

hax merged commit fe74486 into main

hax deleted the api-nextLast branch

July 7, 2022 02:21

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