Skip to content

Comments

sage --fiximports CompGIT/Git.py#23

Merged
jesusmartinez-garcia merged 1 commit intoRobbie-H:mainfrom
passagemath:replace-sage.all-imports
Jul 22, 2025
Merged

sage --fiximports CompGIT/Git.py#23
jesusmartinez-garcia merged 1 commit intoRobbie-H:mainfrom
passagemath:replace-sage.all-imports

Conversation

@mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Jun 25, 2025

In library code, it is best to avoid imports from sage.all.

Here I used the sage --fiximports tool to replace some of these imports by specific imports.

There are more imports from sage.all in this package using from sage.all import *, but the tool cannot take care of those * imports automatically.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants