Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Temporal: Tests for disallowed corner case in rounding durations #4152

Merged

Conversation

ptomato
Copy link
Contributor

@ptomato ptomato commented Jul 15, 2024

This test covers a normative change to disallow an ambiguous corner case in the options bag for Temporal.Duration.prototype.round().

See tc39/proposal-temporal#2916

@ptomato
Copy link
Contributor Author

ptomato commented Jul 30, 2024

Achieved consensus at TC39 plenary 2024-07-30.

@ptomato ptomato added has consensus This has committee consensus and removed awaiting consensus This needs committee consensus before it can be eligible to be merged. labels Jul 30, 2024
Copy link
Contributor

@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.

LGTM

This test covers a normative change to disallow an ambiguous corner case
in the options bag for Temporal.Duration.prototype.round().

See tc39/proposal-temporal#2916
@ptomato ptomato force-pushed the temporal-2902-rounding-increment-calendar branch from 2e178cc to af9f702 Compare September 5, 2024 22:04
@ptomato
Copy link
Contributor Author

ptomato commented Sep 5, 2024

The spec PR is ready to be merged.

@ptomato ptomato merged commit c93ae0a into tc39:main Sep 5, 2024
8 checks passed
@ptomato ptomato deleted the temporal-2902-rounding-increment-calendar branch September 5, 2024 22:08
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.

3 participants