Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Single ruff linter #562

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Single ruff linter #562

wants to merge 2 commits into from

Conversation

ajparsons
Copy link
Contributor

Quick experiment - not essential to anything but a test of how easily ruff fits an existing project.

This PR replaces black, isort and flake8 with ruff - with config settings to mirror the current isort settings.

Linting differences are mostly blank lines (annoying some of which I introduced the reverse with black the other day) - one instance where it changes /B to /b - generally ruff follows black closely.

The advantage of it (apart from condensing a few packages) is it's very very quick. Noticeably much faster than running black even on a smallish project like this - which is nice when running something that formats on save.

- replace black, flake8, isort
- all blank lines except...
- lower case unicode \b in import_air_quality_data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant