Skip to content
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 for flake8 2.5.1 #139

Merged
merged 1 commit into from Feb 1, 2016

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Jan 12, 2016

This should restore the Travis status to passing - see https://travis-ci.org/ome/figure/builds/101965790 for the current state. The alternative is to cap the version of flake8 installed in .travis.yml as we do on openmicroscopy.git. I have no strong preference either way.

@will-moore
Copy link
Member

@sbesson Travis is failing on new PR #140, needing the fix that's in this PR. If you're happy to merge this one, please do.

@sbesson
Copy link
Member Author

sbesson commented Feb 1, 2016

Got 👍 from @joshmoore. Merging to let #140 be reviewed.

sbesson added a commit that referenced this pull request Feb 1, 2016
Fix flake8 warnings for flake8 2.5.1
@sbesson sbesson merged commit 26fd966 into ome:master Feb 1, 2016
@sbesson sbesson deleted the travis_flake8_2.5.1 branch February 1, 2016 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants