PROTON-2322/PROTON-2813: [Python] Finish PEP8 sanitization

[astitcher]

This is a broad but shallow change, that changes a lot of files in fairly minor ways.

[astitcher]

I want to be sure this didn't break any build, and especially any version of python from 3.9 onwards.
But assuming is doesn't then I'll commit it to main.

[jiridanek]

One of the w503, w504 should remain disabled, imo, otherwise it's not possible to have line breaks in expressions, which can be useful. But if it works out for you without this, then I guess it's fine.

[jiridanek]

https://peps.python.org/pep-0008/#should-a-line-break-before-or-after-a-binary-operator

[astitcher]

I rewrote the code to avoid using more than one line (and I think made it a little easier to understand)

And in general it's avoidable if you're checking before committing the change.

[jiridanek]

Tried a Python compatibility check in IntelliJ, and got

qpid-proton/python/tests/proton_tests/main.py

Line 29 in df1e9b6

import cgi

python 3.13 will be removing the cgi module from the standard library, so it needs to be removed from proton examples eventually

https://docs.python.org/3.13/whatsnew/3.13.html#summary-release-highlights

[jiridanek]

don't like this warning, would've kept it disabled

[astitcher]

I agree that I wouldn't put it in a coding standard myself - but forbidding the letter 'l' to avoid confusion eith the number '1' has some sense IMO and it is in PEP8.

Again it's easy enough to avoid if you're checking before commit.

[astitcher]

python 3.13 will be removing the cgi module from the standard library, so it needs to be removed from proton examples eventually

https://docs.python.org/3.13/whatsnew/3.13.html#summary-release-highlights

That's interesting - I'll look at that I wonder what we use it for.