Closed RiesBen closed 7 months ago
Hello @RiesBen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
src/kartograf/filters/ring_changes.py
:Line 121:80: E501 line too long (92 > 79 characters) Line 122:80: E501 line too long (80 > 79 characters) Line 123:80: E501 line too long (84 > 79 characters) Line 144:80: E501 line too long (94 > 79 characters)
src/kartograf/tests/test_ring_filters.py
:Line 104:80: E501 line too long (80 > 79 characters) Line 114:80: E501 line too long (87 > 79 characters) Line 117:80: E501 line too long (80 > 79 characters)
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.35%. Comparing base (
f371334
) to head (875359c
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This branch adds a ring hybridization change filter. (like discussed here: #30)