Skip to content

Conversation

@bajtos
Copy link
Member

@bajtos bajtos commented Apr 17, 2020

Update dependencies

  • async to ^3.2
  • chalk to ^4.0
  • uuid to ^7.0

Upgrade dev dependencies

  • mocha to ^7.1
  • sinon to ^9.0

[SEMVER-MAJOR] Upgrade pg to 8.0

Notable breaking changes:

  • Change default behavior when not specifying rejectUnauthorized with the SSL connection parameters. Previously we defaulted to rejectUnauthorized: false when it was not specifically included. We now default to rejectUnauthorized: true. Manually specify { ssl: { rejectUnauthorized: false } } for old behavior.

  • Change default database when not specified to use the user config option if available. Previously process.env.USER was used.

See pg's changelog for the full list of breaking changes.
https://github.com/brianc/node-postgres/blob/master/CHANGELOG.md#pg800

Checklist

👉 Read and sign the CLA (Contributor License Agreement) 👈

  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • Commit messages are following our guidelines
bajtos added 3 commits April 17, 2020 10:14
- async to ^3.2 - chalk to ^4.0 - uuid to ^7.0 Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
Notable breaking changes: - Change default behavior when not specifying `rejectUnauthorized` with the SSL connection parameters. Previously we defaulted to `rejectUnauthorized: false` when it was not specifically included. We now default to `rejectUnauthorized: true`. Manually specify `{ ssl: { rejectUnauthorized: false } }` for old behavior. - Change default database when not specified to use the `user` config option if available. Previously `process.env.USER` was used. See pg's changelog for the full list of breaking changes. https://github.com/brianc/node-postgres/blob/master/CHANGELOG.md#pg800 Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
- mocha to ^7.1 - sinon to ^9.0 Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
@bajtos bajtos requested a review from dhmlau April 17, 2020 08:22
@bajtos bajtos self-assigned this Apr 17, 2020
Copy link
Contributor

@jannyHou jannyHou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch for the breaking change, LGTM 👍

@bajtos bajtos merged commit 12034b0 into master Apr 20, 2020
@delete-merged-branch delete-merged-branch bot deleted the update-deps branch April 20, 2020 11:49
@bajtos bajtos mentioned this pull request Apr 20, 2020
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants