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

[locale] Fix Indian week start date and week number #5502

Merged
merged 1 commit into from May 19, 2020

Conversation

kumarharsh
Copy link
Contributor

@ichernev - following the directions in previous PR, I'm updating the en-in locale.

The Indian week starts on a Sunday, and 1st January is the first day of the week. This is the most official source I could find:
https://unicode-org.github.io/cldr-staging/charts/37/supplemental/territory_information.html#IN

image

If I understand the "Days in week (min)" column, the value of 1 would also mean that the first week would start on 1st Jan, even if it's a Saturday. Am I correct?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 88.425% when pulling 1b1aa07 on kumarharsh:fix/en-in_weeks into 475a1e0 on moment:develop.

@kumarharsh kumarharsh changed the title Fix Indian week start date and week number locale: Fix Indian week start date and week number May 3, 2020
@kumarharsh kumarharsh changed the title locale: Fix Indian week start date and week number [locale] Fix Indian week start date and week number May 3, 2020
@marwahaha
Copy link
Member

I'm not an expert with the CLDR standard, but I think that makes sense.

Thank you @kumarharsh for your repeated contributions to this library.

@marwahaha marwahaha merged commit 04b275c into moment:develop May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants