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.

Sign up for GitHub

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

Jump to bottom

Linting with ruff #314

Open

akx wants to merge 4 commits into lincolnloop:main from akx:linting

akx commented Feb 15, 2023

This PR:

configures ruff for linting via pre-commit and the GH action for it
fixes some end-of-line/end-of-file issues via pre-commit-hooks
removes every trace of noqa by adding __all__s for those imports
applies the remaining ruff autofixes, which were basically unsorted imports (isort had technically been configured by 179c94d but nothing ran it).

akx added 4 commits

February 15, 2023 16:29


          Configure pre-commit + ruff for linting instead of flake8 (and isort …

2fbb84c

…that wasn't being run)


          Apply end-of-(file/line) fixes

68ef5b3


          Add __all__s to get rid of noqas

eb4c061


          Apply ruff autofixes

e3710eb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment