-
Notifications
You must be signed in to change notification settings - Fork 12
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
Rebuilded pull request from alank64 #10
base: master
Are you sure you want to change the base?
Conversation
linko
commented
Apr 30, 2013
- Rebuilded commits from PR Examples of model and cypher query #7 ,builds the node_id and relationship_id on cypher query results #8
- Added psych dependency ~> 1.3 to run on ruby 1.9.2 (Error: can't dump anonymous class Class)
Will this work? |
Any news on this? |
@linko sorry. I've been really busy the last few weeks! I looks like I may have some time towards the end of this week. :-) |
I already use this code in one of my projects - works well. |
@therubymug Still nothing? |
@linko sorry, your pull request only covers part of what my vision for what this feature is/should be. This is why I haven't merged this into master. Part of my virtus branch is aiming to do what you're doing in this pull request. I just have not had time (been busy with paid work) to finish that branch! Again, sorry about the delay it's just the way it goes sometimes. :-/ |
Rogelio, Maybe others could contribute to your Virtus branch, like if you were to This way we wouldn't waste effort on implementing things that your branch Keymaker is your baby, but just giving suggestions that might make it move On Wed, Jun 26, 2013 at 6:06 PM, Rogelio J. Samour <[email protected]
|
Hey! I'm using it in production already for about 5 months :) Maybe it's time to merge it? Only if it's a part of next big changes/updates. |