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

Add missing values in conversion matrices #720

Merged
merged 1 commit into from
Jun 5, 2020

Conversation

GillesDebunne
Copy link
Collaborator

And a test to check all values are defined.

Note the conversion toObject in the test :

Duration.fromObject({ seconds: Nan }).get('seconds') returns 0, which may be considered a bug.

@icambron icambron merged commit e81600b into moment:master Jun 5, 2020
@icambron
Copy link
Member

icambron commented Jun 5, 2020

Ha! I'd never noticed that.

@GillesDebunne GillesDebunne deleted the fullConversionMatrices branch August 19, 2020 13:25
icambron added a commit that referenced this pull request Sep 28, 2020
* remove console.log call and bump to 1.24.1

* update package-lock.json

* fix Interval.hasSame behaviour for empty intervals (#712)

Fixes #709

* Mark the package as side-effects-free (#713)

Addresses #710

* Add DATE_MED_WITH_WEEKDAY preset (#716)

* All tests assume America/NewYork timezone (#718)

* Add missing values in conversion matrices (#720)

* Remove normalization in ShiftTo (#721)

* RelativeTime test fails on last day of the month (#719)

* Update wording around date equality test (#725)

* Update wording around date equality test

As discussed in #108

* Fix for negative values in plus/minus. Fixes #645 and #669 (#722)

Requires the `fullConversionMatrices` and `noNormalizationInShiftTo` branches
to be merged first.

* Fixed typos in docs where 'month' was used in place of 'weekday' (#732)

* feat: add support for large durations in seconds (#737)

* Bump lodash from 4.17.14 to 4.17.19 (#739)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.14 to 4.17.19.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.14...4.17.19)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Interval creation supports Settings.throwOnInvalid. Fixes #728 (#761)

* formatOffset handles signed offsets. Fixes #665 (#756)

* Update package-lock.json after npm install (#751)

Co-authored-by: Gilles Debunne <gilles.debunne@gmail.com>

* Add tests around DateTime.toISO() (#752)

* Round minutes in timezone offset display. Fixes #724 (#755)

* Support arbitrary precision in ISO milliseconds. Fixes #757 (#758)

* Support arbitrary precision in ISO milliseconds. Fixes #757

* Limit regex to 30 digits to prevent DOS

* fromFormat handles non breakable spaces. Fixes #714 (#762)

* Update Why page, remove outdated Relative references. Fixes #679. (#765)

* bump to 1.25.0

* Introduce DateTime.now() (#766)

* Bump codecov from 3.6.5 to 3.7.1 (#744)

Bumps [codecov](https://github.com/codecov/codecov-node) from 3.6.5 to 3.7.1.
- [Release notes](https://github.com/codecov/codecov-node/releases)
- [Commits](codecov/codecov-node@v3.6.5...v3.7.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update endOf() documentation with the same units as startOf() (#779)

* UnitFormat was folded into Unified NumberFormat (#785)

As per tc39/ecma402#32 (comment)

Co-authored-by: Isaac Cambron <isaac@isaaccambron.com>
Co-authored-by: downace <downace@users.noreply.github.com>
Co-authored-by: Ryota Kameoka <kameoka.ryota@gmail.com>
Co-authored-by: saltire <saltire@users.noreply.github.com>
Co-authored-by: Jon Knowles <jknowles@digitalprimates.net>
Co-authored-by: Jörg Bayreuther <jb@visualjerk.de>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Gilles Debunne <63719587+gdebunne@users.noreply.github.com>
Co-authored-by: Anthon Holmqvist <anthonkendel@users.noreply.github.com>
Co-authored-by: Bart Enkelaar <benkelaar@gmail.com>
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.

2 participants