Closed akamhy closed 3 years ago
Hello @akamhy! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
tests/test_cli.py
:Line 32:89: E501 line too long (132 > 88 characters)
waybackpy/utils.py
:Line 432:89: E501 line too long (107 > 88 characters)
Merging #93 (6755d4d) into master (db8f902) will decrease coverage by
0.87%
. The diff coverage is61.53%
.
@@ Coverage Diff @@
## master #93 +/- ##
==========================================
- Coverage 90.94% 90.06% -0.88%
==========================================
Files 8 8
Lines 563 574 +11
==========================================
+ Hits 512 517 +5
- Misses 51 57 +6
Impacted Files | Coverage Δ | |
---|---|---|
waybackpy/wrapper.py | 82.78% <ø> (ø) |
|
waybackpy/utils.py | 87.27% <50.00%> (-2.54%) |
:arrow_down: |
waybackpy/cli.py | 93.95% <100.00%> (-0.49%) |
:arrow_down: |
waybackpy/exceptions.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 db8f902...6755d4d. Read the comment docs.
…redirect. closes #92