Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Editorial tweaks #48

Merged
merged 3 commits into from
Apr 1, 2025
Merged

Editorial tweaks #48

merged 3 commits into from
Apr 1, 2025

Conversation

linusg
Copy link
Member

@linusg linusg commented Mar 28, 2025

See linked ECMA-262 commits in each commit messages.

@linusg linusg force-pushed the editorial-tweaks branch from 7acbf44 to f1e23be Compare March 28, 2025 00:30
@js-choi
Copy link
Collaborator

js-choi commented Apr 1, 2025

Thank you for the pull request, and thank you for including cross-references to corresponding changes in ECMA-262. Good catch too with the Abstract Closure’s current failure to capture asyncItems.

I think this is good to merge. @nicolo-ribaudo, @ljharb reviewed the spec in the past for pre–Stage 3 (#14); I’ll wait for their approvals at least @nicolo-ribaudo’s approval too.

Copy link
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.

As this is all editorial, you probably can merge without waiting for Jordan's review.

@js-choi js-choi merged commit a64b19b into tc39:main Apr 1, 2025
@linusg linusg deleted the editorial-tweaks branch April 1, 2025 19:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants