Skip to content
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

Fix compatibility with newer versions of node #39

Merged
merged 5 commits into from May 3, 2016
Merged

Conversation

brianc
Copy link
Owner

@brianc brianc commented May 3, 2016

Most of the work was done by @chriskinsman - I just doing a bit of maintainer stuff here w/ regard to enabling more tests & testing more versions.

chriskinsman and others added 5 commits December 16, 2015 16:21
…between on/once and inconsistenly loses unshifted data depending on node version. Instead just split stream and send it to copy stream and the connection.stream at the same time. Disconnecting copy stream just means unpiping. Added handleCopyData to fulfill query contract but ignore the incoming data.

Add node 4.2.2 to Travis
Minimum postgres 9.2 to allow tests to complete in Travis

Remove test that is no longer needed since we no longer disconnect/reconnect listeners
@brianc
Copy link
Owner Author

brianc commented May 3, 2016

Closes #35! Thanks again @chriskinsman and those involved! 👏 🙇

@brianc brianc merged commit c4a0e6d into master May 3, 2016
@brianc
Copy link
Owner Author

brianc commented May 3, 2016

note: because we are dropping support for node@v0.10.0 I'm going to bump the major version of this lib.

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.

None yet

2 participants