-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
switch to ruff #5218
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
switch to ruff #5218
Conversation
@gvwilson big picture comments --
|
af41d30
to
9c60129
Compare
What would you think of adding an item to the pull request template documentations section that recommends: [ ] start Python code blocks with ```python? It's something that is not obvious in the checklist and that I've forgotten to do a couple times. Forgetting to do so can prevent the example from displaying. |
@emilykl clean build - I've put |
- Run `ruff format --check .` instead of `ruff format .` in Circle CI. - Use `base_type is list` instead of `base_type == list`. - Remove commented-out lines of code. - Remove unnecessary parentheses around some strings. - Be more consistent about (re-)naming test functions that had duplicated names. - Use `noqa: F401` to suppress `ruff check` complaint about unused import.
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.
Woohoo! 🚀
requires-optional.txt
andtest_requirements/*
(no longer used).commands.py
to useargparse
instead of hand-rolled.codegen/__init__.py
(all at top level)."dev"
section ofpyproject.toml
to useruff
instead ofblack
.pyproject.toml
to installinflect
andrequests
.uv.lock
file.codegen/__init__.py
to useruff
instead ofblack
.ruff
to reformat existing code (runpython commands.py format
).ruff
to check code (runpython commands.py lint
).This PR has been rebuilt on top of #5214 to include recent changes to validator generation.
This PR currently fails the build because Circle CI is expecting
requires-optional.txt
, which this PR removes.We need to decide if we're leaving build on Circle CI or moving it to GitHub.