Skip to content

Conversation

fcelda
Copy link
Contributor

@fcelda fcelda commented Mar 21, 2012

No description provided.

@Byron
Copy link
Member

Byron commented Jun 7, 2012

Thanks for the fix.

However, without a test I will not be able to merge it in.
Please use nosetest to run the existing tests. You could add a new one which verifies your fix.

Thank you

@fcelda
Copy link
Contributor Author

fcelda commented Jun 7, 2012

The problem was that the tests didn't run on my machine, otherwise I would add the tests as well. Take a look at my previous comment on this: #41 (comment)

I do not remember what was the problem, I have never worked with nose before. But let me take a look again. ;-)

@Byron
Copy link
Member

Byron commented Nov 19, 2014

Thank you !
I believe your fix is already implemented by now.

@Byron Byron closed this Nov 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants