-
Notifications
You must be signed in to change notification settings - Fork 9
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
Adopt Ruff #73
Adopt Ruff #73
Conversation
@chris48s what do you think of adding a separate ruff workflow like https://github.com/torchbox/wagtail-bynder/blob/main/.github/workflows/ruff.yml -- this will add handy inline annotations on the PR should there be failures |
Sounds like a reasonable improvement. It requires us to drop the "delegate all linter management to pre-commit" thing and install our tools in the local environment first, which I do plan to deal with, but I think should be another issue - as stated in the top post. I don't think we need to do all that in this PR for it to be worth merging. |
...ookiecutter.__project_name_kebab }}/{{ cookiecutter.__project_name_snake }}/test/settings.py
Outdated
Show resolved
Hide resolved
{{ cookiecutter.__project_name_kebab }}/.pre-commit-config.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few tidy ups and this is good to go
Co-authored-by: Dan Braghiș <dan@zerolab.org>
…ect_name_snake }}/test/settings.py Co-authored-by: Dan Braghiș <dan@zerolab.org>
Co-authored-by: Dan Braghiș <dan@zerolab.org>
Why not? All the cool kids are doing it 😎
Closes #58
This PR replaces
with ruff, and provides a very similar ruff config to what we previously had for those tools.
In this PR, I have stuck to doing everything with pre-commit hooks and no local installs. I am not making the switch away from hook repos to using "local" here, just to keep the scope constrained. I do plan to do that, but it is another issue for another day.