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
Bump version to 1.1.0 and update CHANGELOG #963
Conversation
Codecov Report
@@ Coverage Diff @@
## master #963 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 10 10
Lines 2014 2014
Branches 323 323
=======================================
Hits 2009 2009
Misses 4 4
Partials 1 1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, a few comments!
CHANGELOG.rst
Outdated
|
||
- [NEW] Added Croatian, Latin, Latvian, Lithuanian and Malay locales. | ||
- [FIX] Internally standardize locales and improve locale validation. | ||
- [FIX] Correct type checking for ``arrow.get(2021, 3, 9)`` construction and internal locale mapping. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to include an example of subclassing a custom locale like Isac mentioned in his PR: #920.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was that available before his PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to Isac it was not, but we can verify on the current stable.
CHANGELOG.rst
Outdated
(<Arrow [2021-04-22T00:00:00+00:00]>, <Arrow [2021-04-28T23:59:59.999999+00:00]>) | ||
|
||
- [NEW] Added Croatian, Latin, Latvian, Lithuanian and Malay locales. | ||
- [FIX] Internally standardize locales and improve locale validation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we put a note that locales should use official ISO notation of the dash rather than underscore, but the change is backward compatible? Or do you think it will just catch on over time with the updated docs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's a good idea to give people a headsup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM one final comment!
@jadchaar all good to go here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Ship it! 🚀
Pull Request Checklist
Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:
tox
ormake test
to find out!).tox -e lint
ormake lint
to find out!).master
branch.If you have any questions about your code changes or any of the points above, please submit your questions along with the pull request and we will try our best to help!
Description of Changes
Bump minor number due to non breaking feature additions.