search

tc39 / proposal-intl-locale-info

An API to expose information of locale, such as week data ( first day of a week, weekend start, weekend end), hour cycle, measurement system, commonly used calendar, etc.
MIT License
56 stars 11 forks source link

Normative: Fix WeekInfoOfLocale #83

Closed FrankYFTang closed 1 month ago

FrankYFTang commented 2 months ago

Fix https://github.com/tc39/proposal-intl-locale-info/issues/82 so so firstDay will return number

@anba @sffc @ben-allen @trflynn89

FrankYFTang commented 2 months ago

@anba @sffc @ben-allen @trflynn89 It will be nice if you all can review and comments before our July 18 , 2024 Thursday TC39 TG 2 meeting.

FrankYFTang commented 1 month ago

@anba in the 2024-07-18 TG2 we discuss and people express concern about the patch you proposed. So I will revert that patch from this PR and open another issue to discuss that part for our next meeting

FrankYFTang commented 1 month ago

I revert @anba's suggestion from the branch and force push it to this PR so the current PR is without that part.

FrankYFTang commented 1 month ago

Remove the "Also fix https://github.com/tc39/proposal-intl-locale-info/issues/30 to explicit set firstDay to "mon" and and minimumDays to 4 if "ca" is "iso8601" " from comments and remove the 8601 handling since it may conflict with the rg issue.

FrankYFTang commented 1 month ago

The PR reach consensus in TG1 2024-7-30 . update the PR and keep the ca / rg resolution vague so the action will not conflict wtih rg. merged

FrankYFTang commented 1 month ago

we may have an ICU bug need to fix about the rg/ca resolution to reflect example on https://unicode.org/reports/tr35/tr35-dates.html#first-day-overrides explicitly what en-AU-u-ca-iso8601-rg-afzzzz-sd-cabc
en-AU-u-ca-iso8601-rg-uszzzz-sd-cabc

should return