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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flake8 warnings and a test generated file #1989

Merged
merged 1 commit into from May 24, 2016

Conversation

Projects
None yet
3 participants
@danlipsa
Contributor

danlipsa commented May 24, 2016

No description provided.

@danlipsa

This comment has been minimized.

Contributor

danlipsa commented May 24, 2016

@danlipsa

This comment has been minimized.

Contributor

danlipsa commented May 24, 2016

@doutriaux1 I missed a baseline and @aashish24 did not flake8 😄 Please review.

@danlipsa danlipsa merged commit 9d06e08 into master May 24, 2016

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
@doutriaux1

This comment has been minimized.

Member

doutriaux1 commented May 24, 2016

@aashish24 I will hold that flake8 mistake as my escape from prison card against you 😉

@doutriaux1 doutriaux1 deleted the dotted-line2 branch May 24, 2016

@aashish24

This comment has been minimized.

Contributor

aashish24 commented May 25, 2016

@danlipsa I fixed these in my other branch (vector scale one), I will rebase.

@doutriaux1 😆

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