Skip to content

Conversation

@alecgibson
Copy link
Collaborator

This change adds support for mongodb v4. The only change
that seems to affect us is:

  • removal of the top-level connect() function

@coveralls
Copy link

coveralls commented Jul 14, 2021

Pull Request Test Coverage Report for Build 1033132494

  • 7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 935995653: 0.0%
Covered Lines: 129
Relevant Lines: 129

💛 - Coveralls

const mongodb = require('mongodb');

module.exports = {
mongodb: mongodb,

Choose a reason for hiding this comment

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

Any particular reason for the extra level of nesting?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

So that we can actively set it in the tests.

This change adds support for [`mongodb` v4][1]. The only [change][2]
that seems to affect us is:

 - removal of the top-level `connect()` function

[1]: https://github.com/mongodb/node-mongodb-native/releases/tag/v4.0.0
[2]: https://github.com/mongodb/node-mongodb-native/blob/4.0/docs/CHANGES_4.0.0.md
@alecgibson alecgibson merged commit ec5c1f2 into master Jul 20, 2021
@alecgibson alecgibson deleted the mongodb-4 branch July 20, 2021 16:24
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