Skip to content

Editorial: clarify that "For each" and "Repeat" steps specify iteration#3822

Merged
ljharb merged 1 commit intomainfrom
for-each-repeat
Apr 10, 2026
Merged

Editorial: clarify that "For each" and "Repeat" steps specify iteration#3822
ljharb merged 1 commit intomainfrom
for-each-repeat

Conversation

@michaelficarra
Copy link
Copy Markdown
Member

@michaelficarra michaelficarra commented Apr 8, 2026

Without mentioning "for each" or "repeat", this paragraph was hard to understand.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

The rendered spec preview for this PR is available as a single page at https://tc39.es/ecma262/pr/3822 and as multiple pages at https://tc39.es/ecma262/pr/3822/multipage .

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

In https://tc39.es/ecma262/#sec-cleanup-finalization-registry we also use "While" instead of "Repeat, while". It's probably better to just reword that rather than explicitly allowing "While" though.

@michaelficarra michaelficarra added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Apr 9, 2026
@michaelficarra
Copy link
Copy Markdown
Member Author

Follow-up: #3829

@ljharb ljharb force-pushed the for-each-repeat branch from 8ebda3a to 4ea0159 Compare April 9, 2026 22:39
@ljharb ljharb merged commit 4ea0159 into main Apr 10, 2026
11 of 12 checks passed
@ljharb ljharb deleted the for-each-repeat branch April 10, 2026 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial change ready to merge Editors believe this PR needs no further reviews, and is ready to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants