some Python cleanups based on PyCharm inspections#2999
Conversation
|
|
ok great.. I can look at it in a few days. |
Thanks. Happy new year! |
I think that the stdout output is ok. Could you please tweak the assertion so it ignores/checks that there is an extra |
393401a to
c718e57
Compare
|
Turns out there was a missing newline in the assert I adjusted. |
|
There is a new failure now ... @danmar Turns out this is also broken on |
54fb5fe to
f3622f8
Compare
|
If Appveyor and Travis are green this can be merged. |
The most interesting thing is that some tests which were not run since there were functions with identical names. I tried to give them proper names on what they do and the commit messages but they could probably get more accurate names.