Update from pep8 to pycodestyle

[karlcow]

see #2825 (comment)
for the context but basically pep8 has been renamed pycodestyle to avoid confusion with the real PEP8.

[karlcow]

pip uninstall pep8
pip install pycodestyle

In CircleCI config we had: pep8 --ignore=E402 webcompat/ tests/ config/secrets.py.example

Reminder: E402 is module level import not at top of file

Let's try a run with pycodestyle --ignore=E402 webcompat/ tests/ config/secrets.py.example

webcompat/helpers.py:417:31: W605 invalid escape sequence '\*'
webcompat/helpers.py:417:33: W605 invalid escape sequence '\*'
webcompat/helpers.py:417:38: W605 invalid escape sequence '\*'
webcompat/helpers.py:417:40: W605 invalid escape sequence '\*'
webcompat/helpers.py:417:42: W605 invalid escape sequence '\:'
webcompat/helpers.py:505:32: W504 line break after binary operator
webcompat/helpers.py:506:31: W504 line break after binary operator
webcompat/helpers.py:507:55: W504 line break after binary operator
webcompat/helpers.py:508:55: W504 line break after binary operator
webcompat/helpers.py:509:30: W504 line break after binary operator
webcompat/helpers.py:510:33: W504 line break after binary operator
webcompat/helpers.py:511:131: W504 line break after binary operator
webcompat/helpers.py:512:75: W504 line break after binary operator
webcompat/helpers.py:513:29: W504 line break after binary operator
webcompat/helpers.py:514:36: W504 line break after binary operator
webcompat/error_handlers/__init__.py:41:42: W504 line break after binary operator
webcompat/error_handlers/__init__.py:42:50: W504 line break after binary operator
webcompat/db/__init__.py:78:1: E305 expected 2 blank lines after class or function definition, found 1
webcompat/api/uploads.py:61:48: W504 line break after binary operator
webcompat/api/uploads.py:116:47: W504 line break after binary operator
tests/unit/test_config.py:56:1: E305 expected 2 blank lines after class or function definition, found 1
tests/unit/test_dashboard.py:90:1: E305 expected 2 blank lines after class or function definition, found 1
tests/unit/test_helpers.py:319:1: E305 expected 2 blank lines after class or function definition, found 1
tests/unit/test_rendering.py:130:1: E305 expected 2 blank lines after class or function definition, found 1
tests/unit/test_tools_changelog.py:63:1: E305 expected 2 blank lines after class or function definition, found 1
tests/unit/test_uploads.py:36:31: W504 line break after binary operator
tests/unit/test_uploads.py:40:37: W504 line break after binary operator
tests/unit/test_uploads.py:44:36: W504 line break after binary operator
tests/unit/test_urls.py:261:80: E501 line too long (81 > 79 characters)
tests/unit/test_webhook.py:291:1: E305 expected 2 blank lines after class or function definition, found 1

Hello modernity, a couple of things to fix :)

We need to choose in between W503 and W504.

pycodestyle --ignore=E402,W504 webcompat/ tests/ config/secrets.py.example

I'm adjusting the documentation too.

[karlcow]

🐝 10:42:48 ~/code/webcompat.com
→ pycodestyle --ignore=E402,W504 webcompat/ tests/ config/secrets.py.example
🐝 10:42:54 ~/code/webcompat.com
→ nosetests
.............................................................................................................
----------------------------------------------------------------------
Ran 109 tests in 2.763s

OK