Skip to content

Normative: Temporal and Intl Era/Month Code stage 4#1044

Open
ptomato wants to merge 16 commits intotc39:mainfrom
ptomato:temporal-stage-4
Open

Normative: Temporal and Intl Era/Month Code stage 4#1044
ptomato wants to merge 16 commits intotc39:mainfrom
ptomato:temporal-stage-4

Conversation

@ptomato
Copy link
Copy Markdown
Contributor

@ptomato ptomato commented Feb 17, 2026

Stage 4 PR for Temporal and Intl Era/Month Code.

Note, the ecmarkup linter is going to fail until ECMA-262 merges Temporal and publishes a new biblio package.

Draft 2026-02-16: I intend to add the stage 4 PR for Intl Era/Month Code in this same PR later. For now, it is up for early feedback.

2026-02-20: Now includes the stage 4 text for Intl Era/Month Code.

@ptomato ptomato marked this pull request as draft February 17, 2026 00:39
@sffc
Copy link
Copy Markdown
Contributor

sffc commented Feb 19, 2026

I'm not certain that this has to move. Will discuss with ECMA-262
editors.
@ptomato ptomato marked this pull request as ready for review February 21, 2026 02:16
@ptomato ptomato changed the title Draft: Normative: Temporal stage 4 Normative: Temporal and Intl Era/Month Code stage 4 Feb 21, 2026
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Feb 21, 2026

Updated with Intl Era/Month Code stage 4 text.

@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Feb 21, 2026

One open question here is whether to refactor NumberFormat to use the GetRoundingIncrementOption and GetRoundingModeOption AOs newly added to ECMA-262.

Especially roundingMode would be a bit of a refactor since we'd have to change things to use spec enums rather than strings. (Of course, this can all be done later.)

The discussion in the ECMA-262 editor call settled on not having the
same GetOption operation as in ECMA-402, so this can stay.
…ndar

See the commit named "(optional) Change approach for specifying
CanonicalizeCalendar" in tc39/ecma262#3759 -
pending discussion with the ECMA-262 editors.
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Feb 25, 2026

I've pushed a small update, keeping all changes in separate commits so it's easy to see what changed. I've reinstated GetOption and CanonicalizeUValue based on discussions in this weeks ECMA-262 editors meeting, and I've incorporated the fix for an assertion failure tc39/proposal-intl-era-monthcode#122.

See the commit named "(review) Express epoch nanoseconds in mathematical
values" in tc39/ecma262#3759 - this change is
based on feedback from the ECMA-262 editors that epoch time should be
specified as mathematical values.
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Mar 6, 2026

Pushed a new commit addressing feedback from @fabon-f (see tc39/proposal-intl-era-monthcode#124)

@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Mar 10, 2026

This adjusts the language in the step in
NonISOMonthDayToISOReferenceDate, clarifying that implementations must
eventually use the month and day results but don't have to calculate
them if the year is already out of range.

h/t fabon-f
See tc39/proposal-intl-era-monthcode#125
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Mar 10, 2026

Pushed a new commit clarifying a sentence in NonISOMonthDayToISOReferenceDate, as per discussion in tc39/proposal-intl-era-monthcode#125. Thanks to @fabon-f for catching.

Copy link
Copy Markdown
Member

@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

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

Some minor issues to fix and some suggestions, but nothing concerning. LGTM!

ptomato added 3 commits March 12, 2026 14:23
Review feedback from Richard.
Review suggestion from Richard.
Review suggestions from Richard.
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Mar 12, 2026

I've pushed a series of commits addressing Richard's and Anba's comments. All are resolved except:

  • The toLocaleStringTimeZone refactor in CreateDateTimeFormat, let's check to see if everyone's satisfied with the change I pushed
  • The comment about [[LocaleData]].[[<Locale>]].[[ca]] which I'm not sure is correct
  • Changing GetDateTimeFormat to a table (or 2 tables), which I can do but would prefer to keep as-is for now
  • <dfn> for "Object for which IsTemporalObject returns true", which I can do but would prefer to keep as-is for now; not sure of a great definition name for "Temporal dot anything except Duration" 😄

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 this pull request may close these issues.

4 participants