Skip to content
This repository was archived by the owner on Jun 15, 2019. It is now read-only.

Conversation

@realityking
Copy link

Fixes #21

There might be a better way but this works 😄

@coveralls
Copy link

coveralls commented Jul 7, 2017

Coverage Status

Coverage increased (+0.001%) to 99.816% when pulling d797a38 on realityking:fix/issue-21 into 6b07944 on jamestalmage:master.

@homerchen19
Copy link

@jamestalmage will this PR be merged?
I face the same problem and hope this PR will solve it.

@jamestalmage
Copy link
Owner

Sorry for being unresponsive on this. Any chance I can get you to write a regression test for this? (one that fails with the old code, and passes with the new).

@realityking
Copy link
Author

Thanks for getting back to this! I forgot all about how this works but I‘ll take a shot at it after Christmas.

@EmolLi
Copy link

EmolLi commented Nov 23, 2018

@realityking Thanks so much for the fix! I have the same bug in my project.
@jamestalmage Can we have this merged? T-T

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants