Skip to content

Temporal: Tests for corner case in TZDB involving DST transition in 1919#4158

Merged
ptomato merged 1 commit intotc39:mainfrom
ptomato:temporal-2910-tzdb-edge-case
Sep 5, 2024
Merged

Temporal: Tests for corner case in TZDB involving DST transition in 1919#4158
ptomato merged 1 commit intotc39:mainfrom
ptomato:temporal-2910-tzdb-edge-case

Conversation

@ptomato
Copy link
Copy Markdown
Contributor

@ptomato ptomato commented Jul 16, 2024

These tests add coverage for a corner case in the TZDB. In spring 1919, the America/Toronto time zone switched to DST at 23:30 on March 30th, skipping an hour ahead to 00:30 on March 31st. This meant that both March 30th and March 31st were 23.5-hour days.

See: tc39/proposal-temporal#2910

@ptomato ptomato force-pushed the temporal-2910-tzdb-edge-case branch from da50409 to 87add44 Compare July 20, 2024 04:54
@ptomato
Copy link
Copy Markdown
Contributor Author

ptomato commented Jul 30, 2024

Achieved consensus at TC39 plenary 2024-07-30.

@ptomato ptomato added the has consensus This has committee consensus label Jul 30, 2024
@ptomato ptomato force-pushed the temporal-2910-tzdb-edge-case branch from 87add44 to 770fd71 Compare July 30, 2024 22:24
@ptomato ptomato removed the awaiting consensus This needs committee consensus before it can be eligible to be merged. label Jul 30, 2024
These tests add coverage for a corner case in the TZDB. In spring 1919,
the America/Toronto time zone switched to DST at 23:30 on March 30th,
skipping an hour ahead to 00:30 on March 31st. This meant that both March
30th and March 31st were 23.5-hour days.

See: tc39/proposal-temporal#2910
@ptomato ptomato force-pushed the temporal-2910-tzdb-edge-case branch from 770fd71 to 3acef77 Compare September 5, 2024 22:15
@ptomato ptomato merged commit d9b1079 into tc39:main Sep 5, 2024
@ptomato ptomato deleted the temporal-2910-tzdb-edge-case branch September 5, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has consensus This has committee consensus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants