Title: Run ruff on Argument Clinic in CI · Issue #110558 · python/cpython · GitHub
Open Graph Title: Run ruff on Argument Clinic in CI · Issue #110558 · python/cpython
X Title: Run ruff on Argument Clinic in CI · Issue #110558 · python/cpython
Description: 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 ...
Open Graph Description: 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/, ru...
X Description: 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/, ru...
Opengraph URL: https://github.com/python/cpython/issues/110558
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Run ruff on Argument Clinic in CI","articleBody":"# Feature or enhancement\n\n### Proposal:\n\nAs @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.\n\nFollowing #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/`.\n\n### Has this already been discussed elsewhere?\n\nThis is a minor feature, which does not need previous discussion elsewhere\n\n### Links to previous discussion of this feature:\n\n_No response_\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-110559\n* gh-110598\n* gh-110603\n* gh-110609\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/AlexWaygood","@type":"Person","name":"AlexWaygood"},"datePublished":"2023-10-09T12:03:35.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/110558/cpython/issues/110558"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:83fc49a4-5959-00e5-7ee5-9eadd104057b |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | A898:3AA710:200103:2BB34F:696A0666 |
| html-safe-nonce | 30eb378bef062bddd7fc4b01ed8f32c34f2207a16b85738b4cd922f95ed19cc7 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBODk4OjNBQTcxMDoyMDAxMDM6MkJCMzRGOjY5NkEwNjY2IiwidmlzaXRvcl9pZCI6IjU4MDkxNDA0ODU5MTY4NTM4NjIiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | d7dbad0c59eebccc4ada872f453ce3acdb2123d068537f99b393759ccc544c32 |
| hovercard-subject-tag | issue:1932915077 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/110558/issue_layout |
| twitter:image | https://opengraph.githubassets.com/342c116f783762ee0f947f02402657889d60959634fe8d2450a02afba61c27bd/python/cpython/issues/110558 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/342c116f783762ee0f947f02402657889d60959634fe8d2450a02afba61c27bd/python/cpython/issues/110558 |
| og:image:alt | 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/, ru... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | AlexWaygood |
| hostname | github.com |
| expected-hostname | github.com |
| None | 578c119ff0247c8b2f2491fbf4fc0395cdf909d4df66598cebdc96ddfc4418dc |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 671c2f67171dbced24284331f3133a613d08c366 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width