Closed akamhy closed 2 years ago
Hello @akamhy! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
waybackpy/cdx_utils.py
:Line 54:9: W503 line break before binary operator Line 55:9: W503 line break before binary operator
waybackpy/cli.py
:Line 281:5: W503 line break before binary operator Line 303:5: W503 line break before binary operator
Merging #158 (e94da0c) into master (f63c6ad) will increase coverage by
1.21%
. The diff coverage is88.09%
.
@@ Coverage Diff @@
## master #158 +/- ##
==========================================
+ Coverage 94.24% 95.45% +1.21%
==========================================
Files 10 10
Lines 608 638 +30
==========================================
+ Hits 573 609 +36
+ Misses 35 29 -6
Impacted Files | Coverage Δ | |
---|---|---|
waybackpy/cdx_api.py | 92.59% <82.35%> (+2.39%) |
:arrow_up: |
waybackpy/cdx_utils.py | 96.29% <88.88%> (-2.15%) |
:arrow_down: |
waybackpy/cli.py | 95.29% <100.00%> (+0.17%) |
:arrow_up: |
waybackpy/exceptions.py | 100.00% <100.00%> (ø) |
|
waybackpy/wrapper.py | 92.53% <0.00%> (+8.95%) |
:arrow_up: |
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 f63c6ad...e94da0c. Read the comment docs.
closes #157