Closed davidorme closed 3 months ago
@alexdewar and @dalonsoa Can you take an initial look at this and see if it fits with your experience of best practice in migrating away from flake8
etc to ruff
.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 94.61%. Comparing base (
9b6cba2
) to head (0b49fc0
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
This PR:
pytest
config to removemypy
andflake8
: runs faster, less cruft.flake8
,isort
,pyupgrade
,black
inpre-commit
withruff
: runs faster, less cruftsetup.cfg
- the new QA chain can all be configured usingpyproject.toml
.ruff
spots butflake8
doesn't.This PR is initially about checking it all works across platforms and rolling out adoption of the new QA chain. Note that the improved runtime applies to local work and the runner speeds for GA workflows.
This PR does not:
Fixes #432
Type of change
Key checklist
pre-commit
checks:$ pre-commit run -a
$ poetry run pytest
Further checks