Skip to content

Editorial: Reword "specified in the ECMA-402 specification" - #3940

Merged
ljharb merged 1 commit into
tc39:mainfrom
ptomato:specified-in-the-specification
Aug 28, 2026
Merged

Editorial: Reword "specified in the ECMA-402 specification"#3940
ljharb merged 1 commit into
tc39:mainfrom
ptomato:specified-in-the-specification

Conversation

@ptomato

@ptomato ptomato commented Aug 5, 2026

Copy link
Copy Markdown
Member

Refer to "ECMA-402", not "the ECMA-402 specification" throughout, as per #3759 (comment)

@nicolo-ribaudo

Copy link
Copy Markdown
Member

There is also a "the ECMA-404 specification" occurrence in https://tc39.es/ecma262/#sec-json-object, could you also update that? (the other occurrences of ECMA-404 are already in the new format)

@ptomato

ptomato commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Done. I should have grepped for ECMA specifications other than 402...

@nicolo-ribaudo nicolo-ribaudo added editor call to be discussed in the next editor call editorial change and removed editor call to be discussed in the next editor call labels Aug 6, 2026

@linusg linusg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would probably s/If an ECMAScript implementation does not include the ECMA-402 API/Otherwise/ to reduce the density of ECMA-402 mentions but this is already an improvement.

@ptomato

ptomato commented Aug 10, 2026

Copy link
Copy Markdown
Member Author

Changed to use the "Otherwise" language.

@gibson042 gibson042 added the request preview ask the bot to trigger a PR preview label Aug 23, 2026
@github-actions github-actions Bot removed the request preview ask the bot to trigger a PR preview label Aug 23, 2026
@github-actions

Copy link
Copy Markdown

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

@gibson042 gibson042 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, but needs a rebase.

@ptomato

ptomato commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

Rebased, I believe it's ready to merge now.

@linusg linusg added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Aug 25, 2026
Refer to "ECMA-402", not "the ECMA-402 specification" throughout,
as per tc39#3759 (comment).
Ditto for ECMA-404.
Change language to "Otherwise" to avoid repetition of "includes the ECMA-402 Internationalization API".
@ljharb
ljharb force-pushed the specified-in-the-specification branch from 9094c69 to ddee48f Compare August 28, 2026 19:33
@ljharb
ljharb merged commit ddee48f into tc39:main Aug 28, 2026
10 checks passed
@ptomato
ptomato deleted the specified-in-the-specification branch August 31, 2026 16:57
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.

5 participants