Enable flake8 rule E225 which checks for missing whitespace around op… #14534

…erators

Lint Python code for rule E225

ACK 74f4e2d648eb40dd2315b46d87d6824f9f419e56 assuming nit is addressed and squash :-)

Hey @practicalswift the nit is fixed and now squashed 👍

<!--e57a25ab6845829454e8d69fc972939a-->Reviewers, this pull request conflicts with the following ones:

• #14296 ([wallet] Remove addwitnessaddress by jnewbery)
• #14066 (gitian-linux: Build binaries for 64-bit POWER by luke-jr)
• #13998 (Scripts and tools: gitian-build.py improvements and corrections by hebasto)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

ACK 0ad199b543465f6aead05ee9014907eca36049e5

Nice to never have to think about or comment on this ever again in future code reviews thanks to this check: review automation FTW :-)

utACK 0ad199b changes are whitespace only (https://github.com/bitcoin/bitcoin/pull/14534/files?w=1), the result is easier to read and the practice is automatically enforced

It would be better to use scripted-diff for this kind of change. Use autopep8 or something else can auto format.

Hey everyone, I did use autopep8 for this.

From the project root directory the command I ran was:

autopep8 --in-place --select=E225 -r .

I would be happy to add other flake8 checks if you want to this pull request.

autopep8 uses pycodestyle

https://pep8.readthedocs.io/en/latest/intro.html#error-codes

Here is the link to autopep8:

https://github.com/hhatto/autopep8

What I mean is writing the commands you were using into the commit message by scripted-diff. Then travis-ci will run the commands again to check if it is the same. So reviewers won't have to run the commands manually on their own computer to check if the change is correct or not.

For example (Haven't tested):

-BEGIN VERIFY SCRIPT-
pip3 install --user autopep8
autopep8 --in-place --select=E225 -r .
-END VERIFY SCRIPT-

Hey @ken2812221 thanks for explaining the process further.

Does that mean you want me to commit again with a scripted diff ?

How can we ensure other pulls don't break master if merged after this?

+1 scripted diff.

Edit: @ken2812221 install in the script?

@jbampton You can git commit --amend to rewrite the commit message and force push it. @promag Yes, I assume travis does not have autopep8 installed. Also we don't want to change travis script. I think this is the only way to install it.

Both variants are allowed in python and the whitespace is not really "missing". Tend to NACK, as I don't see the advantage of doing this code churn for everyone.

See also #14540

<!--cf906140f33d8803c4a75a2196329ecb-->Needs rebase

Trivial pull request still needs rebase. Closing for now.