Meta-ticket: Fix E701, E702, W605 for tox -e pycodestyle-minimal

[kliem]

We fix code style to address code style reports by
pycodestyle, for warnings and error codes:

E701 multiple statements on one line (colon)
E702 multiple statements on one line (semicolon)
W605 invalid escape sequence '\Z'

See:

• https://pycodestyle.pycqa.org/en/latest/intro.html#error-codes

This makes tox -e pycodestyle-minimal pass
with the current configuration.

As this touches many files, we split this into
several smaller tickets.

• fixes for flake8 E70* #31286: fixes for flake8 E70*
• Some E701, E702 (No 1) #31968: Some E701, E702 (No 1)
• Some E701, E702 (No 2) #31969: Some E701, E702 (No 2)
• Fix W605 #31970: Fix W605
• E701, E702 for rings #31971: E701, E702 for rings
• E701, E702 for docs, plot, misc #31972: E701, E702 for docs, plot, misc
• E701, E702 for schemes and trailing empty lines in geometry #31973: E701, E702 for geometry
• E701, E702 for algebras, crypto, quadratic form, structure #31974: E701, E702 for algebras, crypto, quadratic form, structure
• E701, E702 for modular #31975: E701, E702 for modular
• E701, E702 for interfaces #31976: E701, E702 for interfaces
• E701, E702 for groups #31977: E701, E702 for groups
• E701, E702 for databases #31978: E701, E702 for databases
• E701, E702 for categories, schemes, tensor, functions #31979: E701, E702 for categories, schemes, tensor, functions
• E701, E702 for matrix, all.py #31980: E701, E702 for matrix, all.py
• some fixes for pycodestyle linter #32279: some fixes for pycodestyle linter
• more pycodestyle checks #32571: more pycodestyle checks

Related:

• fixes for flake8 E70* #31286: fixes for flake8 E70*
• refine relint configuration and make it pass on combinat #31794: refine relint configuration and make it pass on combinat
• fix patchbot plugin failures for #20970 #30348: fix patchbot plugin failures for Gabidulin Codes #20970

CC: @slel

Component: refactoring

Keywords: pycodestyle

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

[fchapoton]

comment:1

see #31286, waiting since 5 months

[fchapoton]

comment:2

see also #31794

[kliem]

comment:3

Thanks for the quick pointer. I didn't really get anything done yet.

[kliem]

comment:5

I just think it would be nice to have the lint badge green per default.

[kliem]

Branch: u/gh-kliem/fix_pycodestyle_minimal

[kliem]

comment:6

The branch is for reference only.

---

Last 10 new commits:

953cda1	E701, E702 for rings
946105e	E701, E702 for docs, plot, misc
75d2dad	E701, E702 for geometry
443f41e	E701, E702 for algebras, crypto, quadratic form, structure
6f8b610	E701, E702 for modular
b2e92c8	E701, E702 for interfaces
c71914b	E701, E702 for groups
ae3f45e	E701, E702 for databases
0875fc6	E701, E702 for categories, schemes, tensor, functions
a4b0c88	E701, E702 for matrix, all.py

[kliem]

Commit: a4b0c88

[kliem]

comment:8

With those tickets above, tox -e pycodestyle-minimal succeeds for me.

[slel]

comment:9

See discussion at #31968, one of the tickets from this meta-ticket.

[slel]

comment:10

Maybe component "refactoring" is more adapted than "doctest framework".

[sagetrac-git]

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

f91363b

fix stupid mistakes

[sagetrac-git]

Changed commit from a4b0c88 to f91363b

[fchapoton]

Changed branch from u/gh-kliem/fix_pycodestyle_minimal to none

[fchapoton]

Changed commit from f91363b to none

[fchapoton]

comment:13

I removed the branch, as this is a metaticket

[fchapoton]

comment:14

This will probably need one last pass, once all tickets are closed.

[kliem]

comment:15

The branch was just illustrating that things worked, once everything is merged. Sorry it is confusing.

[kliem]

comment:18

With #32571 this ticket can be closed:

https://github.com/sagemath/sagetrac-mirror/actions/runs/1279015500