-
Notifications
You must be signed in to change notification settings - Fork 73
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
is v2 up to date? #156
Comments
Please merge |
A straight merge is not possible. v2 is substantially different. These patches need to be manually reproduced on v2 to land. PR’s are welcome. Sorry for the delay—I will try to grow the contributor base. |
I attempted a merge but there were many conflicts and I had the feeling that I was not able to resolve the conflicts as it would have been expected. I worked around the problem by building an alternative library ( I don't like the idea of publishing duplicates to npm, but in this case (and the frustration of a morning spent tracing a bug caused by montagejs/collections#139) it was the solution with the least effort. |
I've noticed that there have been many commits in v1 and v2 without any merging. Issues like #149 have only been resolved in v1. Is the v2-branch abandoned? Would it make sense to merge v1 into v2 to apply the remaining changes? Or are the differences so large that they cannot be merged?
I want to use v2 now, because of montagejs/collections#139
The text was updated successfully, but these errors were encountered: