-
-
Notifications
You must be signed in to change notification settings - Fork 481
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
Details in interfaces #36758
Details in interfaces #36758
Conversation
src/sage/interfaces/qepcad.py
Outdated
@@ -1930,7 +1934,7 @@ def atomic(self, lhs, op='=', rhs=0): | |||
|
|||
op = self._normalize_op(op) | |||
|
|||
formula = ('%r %s %r' % (lhs, op, rhs)) | |||
formula = (f'{lhs!r} {op} {rhs!r}') |
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.
formula = (f'{lhs!r} {op} {rhs!r}') | |
formula = f'{lhs!r} {op} {rhs!r}' |
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
Documentation preview for this PR (built with commit 6b94f66; 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.
LGTM
Just fixing a few suggestions of `ruff` and `pycodestyle` there ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. URL: sagemath#36758 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe
Just fixing a few suggestions of
ruff
andpycodestyle
there📝 Checklist