Closed jwodder closed 2 weeks ago
Ah thank you, it was clunky and driving me crazy :)
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 86.45%. Comparing base (
7d0fd7a
) to head (7086b75
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
CI fail is just code-cov. I tested it out and it worked as expected.
.update-readme-help.sh
benefits from being rewritten in Python because:sed -i
with no extension differs between Linux and macOS, and thus the current shell script is not portable.Running
.update-readme-help.sh
via pre-commit is problematic, as it requiresduct
to be installed in the current Python environment, yet no provision is made for this. Unless you can come up with a simple, clean way to installduct
in editable mode in some virtualenv, I think it's better to just leave this out of pre-commit.