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

[0.9] travis fix? #3308

Open
wants to merge 2 commits into
base: 0.9
from

Conversation

@nabobalis
Copy link
Contributor

commented Aug 17, 2019

See if the 0.9 CI still works.

TODO:

  • Fix the rest of this

@nabobalis nabobalis force-pushed the nabobalis:0.9 branch 2 times, most recently from 4f94660 to 448e627 Aug 17, 2019

@pep8speaks

This comment has been minimized.

Copy link

commented Aug 17, 2019

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

Line 512:101: E501 line too long (101 > 100 characters)
Line 526:101: E501 line too long (103 > 100 characters)
Line 533:101: E501 line too long (104 > 100 characters)
Line 540:101: E501 line too long (102 > 100 characters)
Line 547:101: E501 line too long (104 > 100 characters)

Comment last updated at 2019-08-19 20:02:53 UTC

@nabobalis nabobalis force-pushed the nabobalis:0.9 branch 2 times, most recently from 57d349c to 5a947ed Aug 17, 2019

@nabobalis nabobalis force-pushed the nabobalis:0.9 branch from 5a947ed to 844eaf2 Aug 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.