-
Notifications
You must be signed in to change notification settings - Fork 96
More streamlining, fault tolerance, and coverage increasing #142
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
Conversation
|
||
def _program_name(self): | ||
def _program_name(self): # pramga: no cover |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pragma, not pramga
@@ -22,8 +22,6 @@ | |||
import pymatbridge as pymat | |||
from .compat import text_type | |||
|
|||
ipython_version = int(IPython.__version__[0]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably safer to use IPython.version_info
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed any more, I used a version-agnostic method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes. This is red. Silly me.
There seems to be an issue with the Travis build. If the tests fail, it waits for 10 minutes and times out. |
@isbadawi, yeah, I'm not sure what to do about that. The Python 3.4 version had the same behavior even when passing. |
We could add a kill switch alarm to make it stop sooner... |
Oh great, now they are all hanging... |
It looks like the test failure leaves it hanging there. On Mon, Feb 16, 2015 at 9:57 AM, Steven Silvester notifications@github.com
|
Hold on before merging, I'm adding two more tests. |
The hanging behaviour was a result of me removing |
More streamlining, fault tolerance, and coverage increasing
87 -> 92%, and caught some bugs!