Skip to content

Rebuilded pull request from alank64 - #10

Open
linko wants to merge 6 commits into
therubymug:masterfrom
linko:node_id_cypher
Open

Rebuilded pull request from alank64#10
linko wants to merge 6 commits into
therubymug:masterfrom
linko:node_id_cypher

Conversation

@linko

@linko linko commented Apr 30, 2013

Copy link
Copy Markdown

@linko

linko commented Apr 30, 2013

Copy link
Copy Markdown
Author

@therubymug, @alank64,

Will this work?

@linko

linko commented May 14, 2013

Copy link
Copy Markdown
Author

Any news on this?

@therubymug

Copy link
Copy Markdown
Owner

@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. :-)

@linko

linko commented Jun 5, 2013

Copy link
Copy Markdown
Author

I already use this code in one of my projects - works well.

@linko

linko commented Jun 26, 2013

Copy link
Copy Markdown
Author

@therubymug Still nothing?

@therubymug

Copy link
Copy Markdown
Owner

@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. :-/

@alank64

alank64 commented Jun 26, 2013

Copy link
Copy Markdown

Rogelio,

Maybe others could contribute to your Virtus branch, like if you were to
outline the direction you want it to go, or the idea your aiming for.

This way we wouldn't waste effort on implementing things that your branch
changes and it would free you a little.

Keymaker is your baby, but just giving suggestions that might make it move
forward faster. ;)

On Wed, Jun 26, 2013 at 6:06 PM, Rogelio J. Samour <notifications@github.com

wrote:

@linko https://github.com/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. :-/


Reply to this email directly or view it on GitHubhttps://github.com//pull/10#issuecomment-20083778
.

@linko

linko commented Sep 16, 2013

Copy link
Copy Markdown
Author

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.

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