Skip to content

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

Closed
AlexWaygood opened this issue Oct 9, 2023 · 0 comments
Closed

Run ruff on Argument Clinic in CI #110558

AlexWaygood opened this issue Oct 9, 2023 · 0 comments
Assignees
Labels
topic-argument-clinic type-feature A feature request or enhancement

Comments

@AlexWaygood
Copy link
Member

AlexWaygood commented Oct 9, 2023

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 the Lib/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) on Tools/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

@AlexWaygood AlexWaygood added type-feature A feature request or enhancement topic-argument-clinic labels Oct 9, 2023
@AlexWaygood AlexWaygood self-assigned this Oct 9, 2023
AlexWaygood added a commit to AlexWaygood/cpython that referenced this issue Oct 9, 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
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
…ent Clinic (GH-110603) (#110609)

gh-110558: Enable ruff's pyupgrade rules when running on Argument Clinic (GH-110603)
(cherry picked from commit fc811c8)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
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
Labels
topic-argument-clinic type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

1 participant