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
some polishing for pbori (ruff and pep8 mostly) #37965
Conversation
Documentation preview for this PR (built with commit 22ed71b; changes) is ready! 🎉 |
def _test(): | ||
import doctest | ||
doctest.testmod() |
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.
delete?
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.
Otherwise LGTM
The failure in test-long is "Random testing has revealed a problem in test_karatsuba_multiplication", unrelated. |
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.
LGTM.
just a basic polish of the ` pbori` folder, fixing pep8 warnings and some ` ruff` suggestions ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#37965 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe
just a basic polish of the ` pbori` folder, fixing pep8 warnings and some ` ruff` suggestions ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#37965 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe
just a basic polish of the
pbori
folder, fixing pep8 warnings and someruff
suggestions📝 Checklist