Closed brynpickering closed 2 days ago
Attention: Patch coverage is 92.30769%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 95.40%. Comparing base (
b7faf2c
) to head (6a08c9e
). Report is 1 commits behind head on main.:exclamation: Current head 6a08c9e differs from pull request most recent head 459ee05
Please upload reports for the commit 459ee05 to get more accurate results.
Please update the CHANGELOG (adding it to the other linting update should be enough, imo).
Fixes all our outdated Python syntax.
Thanks to @irm-codebase for pointing to pyupgrade. I think it's worth having it in our core config as it will keep our code cleaner and easier to review/maintain.
Summary of changes in this pull request
UP
to ruff config.pre-commit
, cleaning up any unresolved fixes.skip-trailing-comma
and add the relevantisort
config to make it work. I much prefer the result, even if it leads to more line changes per commit, on average.Reviewer checklist