Closed richardjgowers closed 1 year ago
Attention: 7 lines
in your changes are missing coverage. Please review.
Comparison is base (
864e3a5
) 92.97% compared to head (0fb145d
) 92.91%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hello @richardjgowers! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
src/kartograf/atom_mapper.py
:Line 290:80: E501 line too long (87 > 79 characters) Line 341:80: E501 line too long (86 > 79 characters) Line 615:80: E501 line too long (91 > 79 characters) Line 616:80: E501 line too long (83 > 79 characters) Line 649:37: E225 missing whitespace around operator Line 649:41: E701 multiple statements on one line (colon) Line 649:80: E501 line too long (89 > 79 characters) Line 695:37: E225 missing whitespace around operator Line 695:41: E701 multiple statements on one line (colon) Line 695:80: E501 line too long (110 > 79 characters)
src/kartograf/filters/ring_changes.py
:Line 84:80: E501 line too long (84 > 79 characters)
also fixed up whole_rings_only filter