node.js client #59

Closed
wants to merge 5 commits into
from

Conversation

Projects
None yet
5 participants

fjakobs commented Feb 13, 2012

fixed merge conflict of #30

fjakobs referenced this pull request Feb 13, 2012

Closed

node.js client #30

Contributor

kastner commented Feb 15, 2012

I'm not sure why this still isn't merge-able... I'll try to take a deeper look.

fjakobs commented Mar 7, 2012

the merge conflict should now REALLY be solved :)

Owner

mrtazz commented Apr 7, 2012

Would you mind splitting the StatsD changes and the client addition into distinct pull requests? Makes testing and integration a lot easier.

Owner

mrtazz commented Jun 8, 2012

Any updates on this?

DTrejo commented Jun 25, 2012

Surprised that this isn't in yet.

Contributor

till commented Jul 13, 2012

Wouldn't mind if the client code was in npm instead.

Owner

mrtazz commented Jul 16, 2012

@till there are already two statsd nodejs clients in npm:

This doesn't mean there can't be another one or anything. Just a general FYI.

Owner

mrtazz commented Jul 16, 2012

I will close this Pull Request for now since there does not seem to be progress to track here. Please don't hesitate to reopen it or open a new one with only the client changes.

mrtazz closed this Jul 16, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment