Skip to content

Conversation

@lunarraid
Copy link

No description provided.

@lunarraid lunarraid closed this Sep 22, 2017
@lazyhoneyant
Copy link

@lunarraid Why do you close this PR? I use your PR well. Is there any problem on this PR?

@lunarraid
Copy link
Author

This PR was made in error. I meant to pull the change into the master branch of my fork only, as this change breaks compatibility with the main repo.

@lazyhoneyant
Copy link

@lunarraid I understand. I've used this PR to my fork repo. Thanks for sharing your useful code.

@Jacse
Copy link

Jacse commented Nov 6, 2017

@lunarraid this is extremely useful and necessary for the library. Do you think you could be convinced to merge your functionality into the lib?

@lunarraid
Copy link
Author

I have no problem with re-opening this if the maintainer is cool with the breaking changes.

@chirag04
Copy link
Owner

chirag04 commented Nov 6, 2017

I'm fine with making a breaking change. Refer the thread here: #124 (comment)

@lunarraid
Copy link
Author

#135 has been opened.

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.

4 participants