Closed systemcatch closed 3 years ago
Merging #924 (7a4c9cd) into master (f15a7ae) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #924 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 1896 1896
Branches 311 311
=========================================
Hits 1896 1896
Impacted Files | Coverage Ξ | |
---|---|---|
arrow/_version.py | 100.00% <100.00%> (ΓΈ) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Ξ = absolute <relative> (impact)
,ΓΈ = not affected
,? = missing data
Powered by Codecov. Last update f15a7ae...ae67704. Read the comment docs.
Hey guys I will make the suggested changes then get this ready for review. Do you think we should just link to the migration guide for breaking changes or do they need to be in the changelog as well?
Hey guys I will make the suggested changes then get this ready for review. Do you think we should just link to the migration guide for breaking changes or do they need to be in the changelog as well?
I think just linking an being explicit about the migration guide outlining the breaking changes is sufficient.
Let's ship it! π π π π π
Feeling excited!
There's no tag on GitHub and no release on PyPI. Is this on purpose? Just wondering :)
There's no tag on GitHub and no release on PyPI. Is this on purpose? Just wondering :)
Haha no, one of us needs to push the button to publish. Should be coming today when I finish work.
@michael-k and it's released! :partying_face:
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
Still a WIP