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

Normative: turn useGrouping false for hours & smaller units when style is 2-digits or numeric #198

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

FrankYFTang
Copy link
Collaborator

Fix #192

Copy link
Collaborator

@ben-allen ben-allen left a comment

Choose a reason for hiding this comment

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

LGTM

@ben-allen ben-allen added consensus We reached a consensus in a discussion meeting, through email or the issue discussion and removed needs-tg1 labels Jul 18, 2024
@FrankYFTang
Copy link
Collaborator Author

I have problem to update this PR to the latest spec text. The spec text got heavily changed by @ben-allen in editorial PR. @ben-allen could you look at your latest spec text and figure out how to address #192 ?

@ben-allen ben-allen reopened this Aug 9, 2024
@ben-allen
Copy link
Collaborator

Updated to set useGrouping to *false* in FormatNumericHour, FormatNumericMinutes, FormatNumericSeconds. @FrankYFTang quick review?

@FrankYFTang
Copy link
Collaborator Author

This PR implemented what the TG2 agreed change and got merged. I think we need someone to bring this PR to the next TG1 to got official consensus for the change inside Stage 3.

@ben-allen
Copy link
Collaborator

ben-allen commented Aug 22, 2024

@FrankYFTang My bad -- I thought we already had TG1 consensus on the previous version. Will add to agenda for next TG1.

@FrankYFTang FrankYFTang mentioned this pull request Sep 25, 2024
@FrankYFTang
Copy link
Collaborator Author

This has reach TC39 consesus on 2024-10-09 meeting

ben-allen added a commit to ben-allen/test262 that referenced this pull request Oct 15, 2024
… unit values while using digital style

Grouping separators suppressed in this situation. See tc39/proposal-intl-duration-format/pull/198
hubot pushed a commit to v8/v8 that referenced this pull request Oct 16, 2024
tc39/proposal-intl-duration-format#198

Bug: 42201655
Change-Id: I12ddd696406efaf9f1b8e849a3ab16251cf741ab
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5807717
Commit-Queue: Frank Tang <ftang@chromium.org>
Reviewed-by: Shu-yu Guo <syg@chromium.org>
Cr-Commit-Position: refs/heads/main@{#96640}
ptomato pushed a commit to ben-allen/test262 that referenced this pull request Oct 30, 2024
… unit values while using digital style

Grouping separators suppressed in this situation. See tc39/proposal-intl-duration-format/pull/198
ptomato pushed a commit to tc39/test262 that referenced this pull request Oct 30, 2024
… unit values while using digital style

Grouping separators suppressed in this situation. See tc39/proposal-intl-duration-format/pull/198
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus We reached a consensus in a discussion meeting, through email or the issue discussion needs-test normative
Projects
None yet
4 participants