Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@khajavi
Copy link
Contributor

@khajavi khajavi commented Jan 8, 2016

…on. #216

@xdarklight
Copy link
Contributor

I implemented a similar fix in #207
Not sure which fix is the better one - but I think the JUnit test I wrote is mandatory to ensure that it doesn't break again

@csokol
Copy link
Contributor

csokol commented Jan 22, 2016

Hi @khajavi, thanks for the fix! I choose to merge #207 since it adds a unit test to this bug as @xdarklight mentioned, but thanks for raising the issue and implementing the fix!

@csokol csokol closed this Jan 22, 2016
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.

3 participants