Skip to content

Update Intl.DurationFormat test with recent default digital style change#3714

Closed
trflynn89 wants to merge 1 commit intotc39:mainfrom
trflynn89:intl_duration_format_update
Closed

Update Intl.DurationFormat test with recent default digital style change#3714
trflynn89 wants to merge 1 commit intotc39:mainfrom
trflynn89:intl_duration_format_update

Conversation

@trflynn89
Copy link
Copy Markdown
Contributor

When the provided style is digital, the default listStyle has been changed from "narrow" to "short". In the "en" locale, this adds commas between the units.

tc39/proposal-intl-duration-format@7495e32

When the provided `style` is digital, the default `listStyle` has been
changed from "narrow" to "short". In the "en" locale, this adds commas
between the units.

tc39/proposal-intl-duration-format@7495e32
@trflynn89 trflynn89 requested a review from a team as a code owner November 5, 2022 00:25
@ptomato ptomato added the awaiting consensus This needs committee consensus before it can be eligible to be merged. label Nov 5, 2022
Copy link
Copy Markdown
Contributor

@ptomato ptomato left a comment

Choose a reason for hiding this comment

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

Approved over at #3713 already. Awaiting consensus in TG1.

@romulocintra
Copy link
Copy Markdown
Member

cc @ryzokuken

@ptomato
Copy link
Copy Markdown
Contributor

ptomato commented Mar 6, 2023

tc39/proposal-intl-duration-format#127 is still awaiting consensus in TC39.

@FrankYFTang
Copy link
Copy Markdown
Contributor

@ryzokuken could you bring tc39/proposal-intl-duration-format#127 to TG1 and TG2 to make sure we can close this issue and land this PR into test262?

@anba
Copy link
Copy Markdown
Contributor

anba commented Aug 24, 2023

This has been fixed in #3895.

@FrankYFTang
Copy link
Copy Markdown
Contributor

We should close this PR. it is dup as @anba mentioned.

@ptomato
Copy link
Copy Markdown
Contributor

ptomato commented Aug 24, 2023

OK, thanks for pointing that out. Also thank you @trflynn89 for submitting it initially.

@ptomato ptomato closed this Aug 24, 2023
@trflynn89 trflynn89 deleted the intl_duration_format_update branch August 25, 2023 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting consensus This needs committee consensus before it can be eligible to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants