Closed jadchaar closed 4 years ago
Merging #823 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #823 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 1757 1776 +19
Branches 300 304 +4
=========================================
+ Hits 1757 1776 +19
Impacted Files | Coverage Δ | |
---|---|---|
arrow/parser.py | 100.00% <100.00%> (ø) |
|
arrow/util.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 e5a8fa6...d855095. Read the comment docs.
Hey Jad, do we want to create regression test based on the bug in #446 ?
assert arrow.get("Tuesday", "dddd").format("dddd") == 'Monday'
@systemcatch I was thinking of creating one, but I did not know where to put it. Do you think it relates more to parsing than formatting?
I think where you've put them is fine, later this evening I can check then approve the PR.
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
Closes: https://github.com/crsmithdev/arrow/issues/446
Replaces https://github.com/crsmithdev/arrow/pull/820 due to GitHub glitch.