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

[mkoeppe]

Part of Meta-ticket #34201: Replace imports from sage.*.all for namespace packages

Also clean up some other outdated importing practices.

CC: @DavidAyotte

Component: refactoring

Author: Matthias Koeppe

Branch/Commit: f76fe41

Reviewer: David Ayotte

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

[mkoeppe]

Description changed:

--- 
+++ 
@@ -1 +1 @@
-
+Also clean up some other outdated importing practices.

[mkoeppe]

Branch: u/mkoeppe/remove_imports_from_sage_rings_all_in_sage_modular

[mkoeppe]

Commit: f76fe41

[mkoeppe]

Author: Matthias Koeppe

[mkoeppe]

New commits:

f76fe41

src/sage/modular: Remove imports from sage.rings.all and .all of other namespace packages

[DavidAyotte]

Reviewer: David Ayotte

[DavidAyotte]

comment:5

It all looks good to me, thanks! May I ask, is there a reason why this was done? Is it part of the modularization process?

[mkoeppe]

Description changed:

--- 
+++ 
@@ -1 +1,3 @@
+Part of Meta-ticket #34201: Replace imports from sage.*.all for namespace packages
+
 Also clean up some other outdated importing practices.

[mkoeppe]

comment:6

Thanks for the review!

Yes, sorry, I should have put it in the ticket description: It is for modularization - see ​https://doc.sagemath.org/html/en/developer/packaging_sage_library.html#module-level-runtime-dependencies

[vbraun]

Changed branch from u/mkoeppe/remove_imports_from_sage_rings_all_in_sage_modular to f76fe41