Skip to content
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

Attempting Patch #101

Merged
merged 1 commit into from
Jan 5, 2018
Merged

Attempting Patch #101

merged 1 commit into from
Jan 5, 2018

Conversation

alokrajiv
Copy link
Collaborator

@alokrajiv alokrajiv commented Nov 30, 2017

Soln proposed by @sum1190

Issue #88

cc @Maziar-Fotouhi

@GillesC can you check all test cases.

@alokrajiv
Copy link
Collaborator Author

@GillesC time to merge?

No-one has complained! :)

@GillesC
Copy link
Collaborator

GillesC commented Jan 5, 2018 via email

@alokrajiv alokrajiv merged commit 8e6e6e7 into master Jan 5, 2018
@ghost
Copy link

ghost commented Feb 8, 2018

@alokrajiv @GillesC Thanks for your efforts.
Is there going to be an official release with this fix?

@alokrajiv
Copy link
Collaborator Author

@Maziar-Fotouhi I think soon we will have to push a new release.

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.

2 participants