Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Apr 24, 2023

📚 Description

Fixes #35557

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@github-actions
Copy link

Documentation preview for this PR is ready! 🎉
Built with commit: dcd1f18

Copy link
Collaborator

@kwankyu kwankyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works well on sage with python 3.9.

@kwankyu
Copy link
Collaborator

kwankyu commented Apr 26, 2023

Nothing related with this PR, but out of curiosity, do you use the number to search for labels like "critical / 2"?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 26, 2023

I think I use it about half of the time. Pretty rare event to use priority labels (in particular, Volker's merging script does not pay attention to them), so it's not hardwired in my fingers yet.
Also typing "s:" to see the status labels has not become a proper habit yet. I find myself typing "need" or "revi" and then click to change labels.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 26, 2023

Thanks for the review!

@kwankyu
Copy link
Collaborator

kwankyu commented Apr 26, 2023

Also typing "s:" to see the status labels has not become a proper habit yet. I find myself typing "need" or "revi" and then click to change labels.

I see. I just scroll to the label in the list :-)

Thanks.

@vbraun vbraun merged commit 9185bef into sagemath:develop Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sage.quadratic_forms: Fix use of staticmethod for Python < 3.10
3 participants