sagemath/sage

Remove imports from sage.rings.all in sage.schemes

Closed this issue · 14 comments

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

Branch pushed to git repo; I updated commit sha1. New commits:

0a5c6f8git grep -l 'all import' src/sage/schemes | xargs sed -i.bak 's/[.]all import Rings$/.rings import Rings/'

Changed commit from 0a5c6f8 to a68b701

Branch pushed to git repo; I updated commit sha1. New commits:

a68b701src/sage/schemes: Remove some .all imports

Branch pushed to git repo; I updated commit sha1. New commits:

cc5d6dasrc/sage/schemes: Remove some more .all imports

Changed commit from a68b701 to cc5d6da

Author: Matthias Koeppe

comment:6

Green bots, ready for review

Description changed:

--- 
+++ 
@@ -1 +1 @@
-
+Part of Meta-ticket #34201.
comment:7

LGTM.

Reviewer: Kwankyu Lee

comment:8

Thank you!