sagemath/sage

Remove imports from sage.interfaces.all

Closed this issue · 13 comments

Depends on #32989

CC: @fchapoton

Component: refactoring

Author: Matthias Koeppe

Branch/Commit: 584d121

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/33007

Commit: 607316f

New commits:

a9b7517git grep -l 'interfaces.all import' | xargs sed -E -i.bak 's/interfaces[.]all import ([a-z][a-z]*)/interfaces.\1 import \1/'
607316fRemove remaining imports from sage.interfaces.all

Author: Matthias Koeppe

Changed commit from 607316f to 0e814dc

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

2d9a84egit grep -l 'misc.all import' | xargs sed -i.bak 's/misc.all import tmp_/misc.temporary_file import tmp_/'
5a8d76eRemove imports from sage.misc.all
97b4dfdgit grep -l 'misc.all import' | xargs sed -i.bak 's/misc.all import sage_eval/misc.sage_eval import sage_eval/'
e72dc8eRemove more imports from sage.misc.all
d114096git grep -l 'misc.all import' | xargs sed -i.bak 's/misc.all import cartesian_product_iterator/misc.mrange import cartesian_product_iterator/'
1ab9441Remove remaining imports from sage.misc.all
0e814dcMerge #32989

Dependencies: #32989

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

37789fdMerge tag '9.5.beta9' into t/33007/remove_imports_from_sage_interfaces_all
584d121src/sage/rings/polynomial/multi_polynomial_libsingular.pyx: Fix import

Changed commit from 0e814dc to 584d121

Reviewer: Frédéric Chapoton

comment:8

ok, ok

comment:9

Thanks.