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

Iterator should in ES* files #266

Closed
skyclouds2001 opened this issue Oct 6, 2024 · 2 comments · Fixed by #272
Closed

Iterator should in ES* files #266

skyclouds2001 opened this issue Oct 6, 2024 · 2 comments · Fixed by #272

Comments

@skyclouds2001
Copy link

skyclouds2001 commented Oct 6, 2024

currently Iterator is in https://github.com/sindresorhus/globals/blob/main/data/worker.mjs#L136, https://github.com/sindresorhus/globals/blob/main/data/nodeBuiltin.mjs#L32, but not in any of the ES* file

it is a JavaScript interface available in all scope, so it is better to place it in ES* file

also, it is not in browser.mjs or serviceworker.mjs currently

@skyclouds2001 skyclouds2001 changed the title Iterator should in ES* files than worker file Iterator should in ES* files Oct 6, 2024
@fisker
Copy link
Collaborator

fisker commented Oct 9, 2024

"Iterator Helpers" reached stage 4 yesterday, but the spec has not include Iterator yet.

@fisker
Copy link
Collaborator

fisker commented Oct 9, 2024

Guess this will get merged soon. tc39/ecma262#3395

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants