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 #187

Merged
merged 2 commits into from Dec 15, 2018

Conversation

Projects
None yet
1 participant
@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 adjustments for Python3 compatibility which is nice 馃憤

N.B. : this PR is on top of #180
N.B. : this changes MoulinetteError inheritance from StandardError to Exception

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 :

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

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
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