Skip to content

Conversation

@booo
Copy link
Contributor

@booo booo commented Aug 1, 2012

Seems like all tests are fine. Hope this is something you can work with.

Best Regards
Philipp

booo added 3 commits August 1, 2012 18:48
TODO: add integration test
TODO: Fix the javascript binary parser too.
We don't parse date types in the javascript binary parser. Keep this in mind if
you plan to use the binary javascript parser.
@mren
Copy link

mren commented Aug 3, 2012

awesome :)

@mren
Copy link

mren commented Aug 3, 2012

i owe you a beer =)

@brianc
Copy link
Owner

brianc commented Aug 7, 2012

@booo you are awesome. If you're interested you can email me at brian.m.carlson@gmail.com and we could discuss you having commit access & npm publish rights for node-postgres. If you'd rather work on your branch and manage through pull requests I'm 100% fine with that as well. Honestly it will take more back & forth communication to share the main repo, but you've more than "earned your stripes" or whatever the hell that means.

@brianc brianc merged commit bab01ea into brianc:master Aug 7, 2012
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