-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
Run ruff on Argument Clinic in CI #110558
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
Labels
Comments
AlexWaygood
added a commit
to AlexWaygood/cpython
that referenced
this issue
Oct 9, 2023
AlexWaygood
added a commit
that referenced
this issue
Oct 10, 2023
AlexWaygood
added a commit
to AlexWaygood/cpython
that referenced
this issue
Oct 10, 2023
AlexWaygood
added a commit
to AlexWaygood/cpython
that referenced
this issue
Oct 10, 2023
AlexWaygood
added a commit
to AlexWaygood/cpython
that referenced
this issue
Oct 10, 2023
hugovk
pushed a commit
that referenced
this issue
Oct 10, 2023
AlexWaygood
added a commit
that referenced
this issue
Oct 10, 2023
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Oct 10, 2023
…nt Clinic (pythonGH-110603) (cherry picked from commit fc811c8) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
AlexWaygood
added a commit
that referenced
this issue
Oct 10, 2023
Glyphack
pushed a commit
to Glyphack/cpython
that referenced
this issue
Sep 2, 2024
Glyphack
pushed a commit
to Glyphack/cpython
that referenced
this issue
Sep 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Feature or enhancement
Proposal:
As @erlend-aasland and I have worked on adding type hints to Argument Clinic over the last few months, and applied various other modernisations to
Tools/clinic/
, running pyflakes on the code on a regular basis has caught numerous small bugs that periodically crept in due to a refactoring. Pyflakes has an extremely low number of false positives, so it would be great to have this run as part of CI to catch these issues before they're merged in.Following #109161, we now run
ruff
in CI on theLib/test/
directory, and ruff has implementations of all the pyflakes error codes. I propose that we add a CI check that runs the full set of pyflakes checks (via ruff) onTools/clinic/
.Has this already been discussed elsewhere?
This is a minor feature, which does not need previous discussion elsewhere
Links to previous discussion of this feature:
No response
Linked PRs
The text was updated successfully, but these errors were encountered: