cython-lint for imports in rings (mostly in padics) #35721

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.

Sign up for GitHub

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

Jump to bottom

Merged

vbraun merged 3 commits into sagemath:develop from fchapoton:cython-lint-in-padics

Jun 11, 2023

Contributor

fchapoton commented Jun 4, 2023 •

edited

Loading

📚 Description

fix remaining cython-lint warnings about unused imports in rings

📝 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

fchapoton added 3 commits

June 4, 2023 09:49


          cython-lint for imports in rings (mostly in padics)

b47a16c


          add back import

69d298f


          undo addition of sig_off

f1546c0

github-actions bot commented Jun 4, 2023

Documentation preview for this PR (built with commit f1546c0) is ready! 🎉

fchapoton added s: needs review t: refactoring labels

mkoeppe approved these changes

View reviewed changes

vbraun merged commit 9c83f60 into sagemath:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels