The Wayback Machine - https://web.archive.org/web/20200906045359/https://github.com/brianc/node-postgres/pull/1304/
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

Remove fallbacks for unsupported Node versions #1304

Merged
merged 6 commits into from Jun 8, 2017

Conversation

@charmander
Copy link
Collaborator

charmander commented May 29, 2017

Part of #1298. There are a few more cleanups made possible with Node 4 (arrow functions instead of .bind(this), startsWith, etc.), but this is a tentative start.

@charmander charmander added this to the [email protected] milestone May 29, 2017
@charmander charmander requested a review from brianc May 29, 2017
@brianc brianc changed the base branch from master to 7.0 Jun 8, 2017
@brianc
brianc approved these changes Jun 8, 2017
Copy link
Owner

brianc left a comment

Sorry for the delay on this. This is perfect and you are awesome! 🤗 🤗 I've re-targeted this at the 7.0 branch. That way I can work there until all the 7.0 milestone work is complete. First commit on 7.0 goes to you!

@brianc brianc merged commit 934ca3a into brianc:7.0 Jun 8, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
brianc added a commit that referenced this pull request Jun 9, 2017
* Add client connectionString tests (#1310)

* Remove redundant tests

* Add client connectionString test

Add test to ensure { connectionString } is respected as an argument to the client constructor

* Add test for connection string property

Also fixed some legacy require statements.

* Normalize native error properties

Map native error properties to the same property names we use for errors from the JS driver.

Fixes #972
Fixes #938
brianc added a commit that referenced this pull request Jun 9, 2017
* Remove unsupported Node versions 0.10 and 0.12 from CI

* Replace deprecated Buffer constructor with .from/.alloc

* Remove Promise polyfill

* Make use of Object.assign

* Remove checks for versions of Node earlier than 4

* Remove Buffer#indexOf fallback for Node 0.10
brianc added a commit that referenced this pull request Jun 9, 2017
* Add client connectionString tests (#1310)

* Remove redundant tests

* Add client connectionString test

Add test to ensure { connectionString } is respected as an argument to the client constructor

* Add test for connection string property

Also fixed some legacy require statements.

* Normalize native error properties

Map native error properties to the same property names we use for errors from the JS driver.

Fixes #972
Fixes #938
@charmander charmander deleted the charmander:remove-node-0.12-support branch Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.