Closed akamhy closed 3 years ago
Hello @akamhy! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
tests/test_cli.py
:Line 102:89: E501 line too long (89 > 88 characters) Line 124:89: E501 line too long (89 > 88 characters) Line 146:89: E501 line too long (89 > 88 characters) Line 168:89: E501 line too long (89 > 88 characters) Line 195:89: E501 line too long (89 > 88 characters) Line 215:89: E501 line too long (89 > 88 characters) Line 235:89: E501 line too long (89 > 88 characters) Line 255:89: E501 line too long (89 > 88 characters) Line 275:89: E501 line too long (89 > 88 characters)
waybackpy/cli.py
:Line 113:89: E501 line too long (107 > 88 characters) Line 165:89: E501 line too long (116 > 88 characters) Line 217:89: E501 line too long (95 > 88 characters)
waybackpy/wrapper.py
:Line 252:89: E501 line too long (107 > 88 characters) Line 257:89: E501 line too long (104 > 88 characters)
Merging #47 (e306fb0) into master (fde28d5) will not change coverage. The diff coverage is
97.56%
.
@@ Coverage Diff @@
## master #47 +/- ##
=======================================
Coverage 92.92% 92.92%
=======================================
Files 5 5
Lines 297 297
=======================================
Hits 276 276
Misses 21 21
Impacted Files | Coverage Δ | |
---|---|---|
waybackpy/exceptions.py | 100.00% <ø> (ø) |
|
waybackpy/cli.py | 95.10% <97.22%> (ø) |
|
waybackpy/wrapper.py | 90.07% <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 fde28d5...e306fb0. Read the comment docs.
First time using black.