Skip to content

[SEMVER-MAJOR]: upgrade dependencies #432

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

Merged
merged 3 commits into from
Apr 20, 2020
Merged

[SEMVER-MAJOR]: upgrade dependencies #432

merged 3 commits into from
Apr 20, 2020

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š <[email protected]>
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š <[email protected]>
- mocha to ^7.1
- sinon to ^9.0

Signed-off-by: Miroslav Bajtoš <[email protected]>
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants