nipreps / nirodents

Apache License 2.0
12 stars 3 forks source link

MAINT: Run black on the full repo, address pep8 errors #27

Closed oesteban closed 4 years ago

oesteban commented 4 years ago

This is a just so you know kind of PR.

pull-assistant[bot] commented 4 years ago
Score: 0.91

Best reviewed: commit by commit


Optimal code review plan

     MAINT: Run black on the full repo, address pep8 errors      fix: revise workflow syntax

Powered by Pull Assistant. Last update 97b13f7 ... 94841e2. Read the comment docs.

pep8speaks commented 4 years ago

Hello @oesteban! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 123:57: E203 whitespace before ':' Line 193:27: E203 whitespace before ':' Line 210:36: E203 whitespace before ':' Line 310:57: E203 whitespace before ':'

Line 152:36: E203 whitespace before ':'

Line 1005:27: E203 whitespace before ':' Line 1022:36: E203 whitespace before ':' Line 1122:57: E203 whitespace before ':' Line 1196:57: E203 whitespace before ':'

Comment last updated at 2020-05-11 17:53:38 UTC
oesteban commented 4 years ago

okay, I'm going to merge. I've opened #28, which surfaced after I addressed all formal problems.