Remove imports from sage.rings.all in sage.schemes
Closed this issue · 14 comments
mkoeppe commented
Part of Meta-ticket #34201.
CC: @nbruin
Component: refactoring
Author: Matthias Koeppe
Branch/Commit: cc5d6da
Reviewer: Kwankyu Lee
Issue created by migration from https://trac.sagemath.org/ticket/34190
mkoeppe commented
sagetrac-git commented
Commit: 0a5c6f8
sagetrac-git commented
Branch pushed to git repo; I updated commit sha1. New commits:
0a5c6f8 | git grep -l 'all import' src/sage/schemes | xargs sed -i.bak 's/[.]all import Rings$/.rings import Rings/' |
sagetrac-git commented
sagetrac-git commented
Branch pushed to git repo; I updated commit sha1. New commits:
a68b701 | src/sage/schemes: Remove some .all imports |
sagetrac-git commented
Branch pushed to git repo; I updated commit sha1. New commits:
cc5d6da | src/sage/schemes: Remove some more .all imports |
sagetrac-git commented
mkoeppe commented
Author: Matthias Koeppe
mkoeppe commented
comment:6
Green bots, ready for review
mkoeppe commented
Description changed:
---
+++
@@ -1 +1 @@
-
+Part of Meta-ticket #34201.kwankyu commented
comment:7
LGTM.
kwankyu commented
Reviewer: Kwankyu Lee
mkoeppe commented
comment:8
Thank you!
vbraun commented
Changed branch from u/mkoeppe/remove_imports_from_sage_rings_all_in_sage_schemes to cc5d6da