New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autopep8 #600

Merged
merged 2 commits into from Dec 15, 2018

Conversation

Projects
None yet
2 participants
@alexAubin
Copy link
Member

alexAubin commented Dec 13, 2018

The problem

Pep8 complains about quite a lot of things

Solution

Ran autopep8 -i --ignore=E501 -a -a -a -a -a *.py */*.py
(Made a few manual tweaks because a few changes were made which werent so logical o.O)

Turns out autopep8 also makes some adjustements for Python3 compatibility which is nice 馃憤

N.B. : this PR is on top of #574

PR Status

Not tested but most of those are only whitespace changes

How to test

...

Validation

  • Principle agreement 0/2 :
  • Quick review 0/1 :
  • Simple test 0/1 :
  • Deep review 0/1 :
@Psycojoker

This comment has been minimized.

Copy link
Member

Psycojoker commented Dec 14, 2018

Can be merged as a microdecision onces the others PR have been merged.

@alexAubin alexAubin merged commit 0311289 into stretch-unstable Dec 15, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@alexAubin alexAubin deleted the autopep8 branch Dec 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment