-
-
Notifications
You must be signed in to change notification settings - Fork 648
add checks to the linter once again #40208
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
Conversation
@tobiasdiez : ready for review, I hope |
Documentation preview for this PR (built with commit 79b4737; changes) is ready! 🎉 |
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.
Sorry, should have caught it in the other PR.
Should we remove the config in the tox files? It seems very redundant and only adds to the maintenance burden. The "maximal" ruff config is also specified in the pyproject.toml
file in the root.
should we add that to the CI-fix tag ? |
follow up of sagemath#40122 about using ruff to check code formatting in the lint CI ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#40208 Reported by: Frédéric Chapoton Reviewer(s): Tobias Diez
sagemathgh-40208: add checks to the linter once again follow up of sagemath#40122 about using ruff to check code formatting in the lint CI ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#40208 Reported by: Frédéric Chapoton Reviewer(s): Tobias Diez
sagemathgh-40208: add checks to the linter once again follow up of sagemath#40122 about using ruff to check code formatting in the lint CI ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#40208 Reported by: Frédéric Chapoton Reviewer(s): Tobias Diez
sagemathgh-40208: add checks to the linter once again follow up of sagemath#40122 about using ruff to check code formatting in the lint CI ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#40208 Reported by: Frédéric Chapoton Reviewer(s): Tobias Diez
sagemathgh-40208: add checks to the linter once again follow up of sagemath#40122 about using ruff to check code formatting in the lint CI ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#40208 Reported by: Frédéric Chapoton Reviewer(s): Tobias Diez
follow up of #40122
about using ruff to check code formatting in the lint CI
📝 Checklist