src/tox.ini: Check that there are no .all imports from namespace packages

[mkoeppe]

We add the following test to the relint linting workflow: Library code should not import from sage.PAC.KAGE.all when sage.PAC.KAGE is a namespace package (see #32501)

[mkoeppe]

New commits:

​7c7e3d2

src/.relint.yml: Check that there are no .all imports from namespace packages

[klee]

It works well. I am positive with the branch.

One small comment: I am dimly aware that there was a discussion somewhere(perhaps on sage-devel) on using two space characters (vs. one space character) to separate sentences. I don't remember the conclusion, but I feel uncomfortable with it, just like a typo. Hence please fix:

 - name: 'namespace_pkg_all_import: import from .all of a namespace package'
   hint: |
     Sage library code should not import from sage.PAC.KAGE.all when sage.PAC.KAGE is an implicit
-    Hint: namespace package.  Type import_statements("SOME_IDENTIFIER") to find a more specific import.
+    Hint: namespace package. Type import_statements("SOME_IDENTIFIER") to find a more specific import.
   pattern: 'from\s+sage[.](categories|misc|rings|combinat|graphs|interfaces|libs)[.]all\s+import'
   filePattern: '.*[.](py|pyx)$'

[git]

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

​928561e

src/.relint.yml: Use 1 space after sentence end

[klee]

Thank you.

[chapoton]

Imho, it is a very bad idea to introduce a check before it has been fully fixed. This just makes the linter useless.

[mkoeppe]

Useless, I don't think so. Sorry for ruining the green linter checkmarks on the tickets though

[gh-tobiasdiez]

I have to agree. Now the linter workflow is falling for each ticket and it's no longer clear if that's because the ticket introduced some new issues or because of the all imports.

I propose to either remove the all imports soon or demote this check to a warning instead of an error (if that's possible with relint).

[mkoeppe]

Replying to gh-tobiasdiez:

I propose to either remove the all imports soon

+1, help welcome

[gh-tobiasdiez]

Sorry, but if you don't plan to work on this yourself I would say we should demote it for now to a warning and fully activate it later.

[mkoeppe]

Replying to gh-tobiasdiez:

... but if you don't plan to work on this yourself ...

Work is already underway on this, see tickets #32989, #32999, #33007, #33146, #33199, #33202