Skip to content

Conversation

p1c2u
Copy link
Contributor

@p1c2u p1c2u commented Oct 31, 2018

Little code cleanup with PEP8

@spulec
Copy link
Owner

spulec commented Nov 4, 2018

Thanks for this.

It looks like one of the tests is now failing. Not positive why, but if we can get that passing, I'll be happy to merge.

@p1c2u
Copy link
Contributor Author

p1c2u commented Nov 6, 2018

@spulec this is failing test from master
https://travis-ci.org/spulec/PyQS/jobs/414318577#L605

There are many tests that are failing on mastet.
This patch fixes one test but others are broken still. You use --stop param for nosetest which stops executing tests after finding broken one.

@p1c2u p1c2u mentioned this pull request Nov 7, 2018
@spulec spulec merged commit dbaa391 into spulec:master Nov 11, 2018
@spulec
Copy link
Owner

spulec commented Nov 11, 2018

Thanks!

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.

2 participants