[misc] Fix type of WeekSpec #4634
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You can set `dow` without setting `doy`, so the type should reflect that
|
This is a really minor change, so I believe that the travis failure isn't related to it. How can I rerun this? |
|
Thanks for opening this request. What is the behavior when you set From my understanding, the typings should reflect intended usage. |
|
To be completely honest, I'm not really sure what really the |
GmaMontero
added a commit
to despegar/moment
that referenced
this pull request
Sep 4, 2019
* [misc] release process fixes (moment#4961) * Fix locale abbreviation in comment * Update package-lock.json * [misc] Fix type of WeekSpec (moment#4634) * Fix "the the" typo in test (moment#4578) * [locale] Fix moment#4568: Insert punjabi for "next" (moment#4583) * Fix type of WeekSpec You can set `dow` without setting `doy`, so the type should reflect that * [new locale] (oc-lnc) Occitan (moment#4957) * Add oc-lnc files * Fix calendar test * Fix the rest of tests * Add locale author * Dont make changes to package-lock * Add newline at end of file * [bugfix] Fix moment#4962 updateLocale with null to reset (moment#4966) * [locale] es: Add es translation for invalidDate (moment#4560) * Add es translation for invalidDate * Fix lint * [locale] improve mk translation (moment#4958) * Update mk.js * Update mk.js * Update mk.js * Update mk.js * [locale] improve mk translation * [tests] Fix various text typos (moment#4976) * [locale] mr (Marathi) (moment#4990) * tests according to changes in mr.js * [locale] mr (Marathi) We Marathi people follow standards mentioned below, to label timespans. Label | Time Span ------------ | ------------- पहाटे (post midnight, pre-sunrise) | >= 12am to < 6am सकाळी (post sunrise, pre-noon) | >= 6am to < 12pm दुपारी (afternoon) | >= 12pm to < 5pm सायंकाळी (evening) | >= 5pm to < 8pm रात्री (night) | >= 8pm to < 12am Actually, 12am is confusing in Marathi tradition. In modern clock system, Moment 12am is considered in arriving date as 0am. And in Marathi tradition it is referred to as final moment of the previous date (Sort of, 0:01 is starting minute of the arriving date) . To comply with modern clock system, I propose to refer to 0am as पहाटे , so that it is considered as moment in the arriving date. Hope this discription itself is not confusing :) Another point is about range of पहाटे, ie. >= 0am to < 6am. Though conventionally, upto 2 am, we refer to the time as रात्री or पहाटे, both alternatively, I want to specifically call पहाटे for early times in the arriving date and रात्री for late times in the departing date, to make it more specific and well-defined, avoiding confusion about date of the time. * [locale] correct translation of locale for Vietnam (moment#4874) * correct translation of locale for Vietnam * correct translation of locale for Vietnam (edit proper src files) * Update author comment line * [locale] (en-sg) Rename locale en-SG to en-sg (moment#5025) * [locale] typos in Irish locale for months (moment#5127) Fixed some typos in the Irish locale regarding months. * [locale] (br) Breton move from 12-hour to 24-hour notation (moment#4975) * Breton : move from 12-hour to 24-hour notation moment#4974 moment#4974 * Replace [e] by : * [locale] bn: Improve translation as per Bangla Academy (moment#5055) * Delete unused locales * Remove unused locales * Update test and build * Add untracked locale
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
You can set
dowwithout settingdoy, so the type should reflect that