Skip to content

Conversation

@alshakero
Copy link
Collaborator

@alshakero alshakero commented Mar 14, 2017

And add Travis badge.

To comply with #5 (comment)

@alshakero alshakero requested review from tomalec and warpech March 14, 2017 12:37
@warpech
Copy link
Contributor

warpech commented Mar 16, 2017

I don't know what the commit 2.0.0 is doing in this branch, but anyway the PR looks GTM :)

@alshakero
Copy link
Collaborator Author

@warpech now that I think about it, this is a non-breaking change, and we didn't change the name. Should I bump to 1.0.2 instead of 2.0.0? I did this because we agreed to bump major to all repo name changes. But this one's name didn't change.

Would appreciate your advice.

@warpech
Copy link
Contributor

warpech commented Mar 21, 2017

Sorry, I have missed the question. Seems that you have already decided 12 days ago: https://github.com/Palindrom/JSON-Patch-OT/releases/tag/2.0.0

@alshakero
Copy link
Collaborator Author

alshakero commented Mar 21, 2017

@warpech no issue at all. I released because I initially thought it's the same situation as this.

@tomalec tomalec deleted the Remove-dist-and-uglification-code branch August 5, 2019 13:39
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