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

feat: add [email protected] support #671

Merged
merged 1 commit into from
Jan 19, 2022
Merged

Conversation

alias-mac
Copy link
Contributor

@alias-mac alias-mac commented Jan 17, 2022

BREAKING CHANGE: it is hard to support both versions (3.x and 4.x) of
native driver on the same branch because 4.x has native types
definitions and they diverge from the ones provided by @types/mongodb.

This means that this version might still be compatible with previous
versions of mongodb native driver (3.6.x and 3.7.x), but we aren't
testing them due to typescript mismatch.

Please open an issue and we will try to support your use case.

@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #671 (b1675be) into master (707d3b6) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #671   +/-   ##
=======================================
  Coverage   97.26%   97.26%           
=======================================
  Files           1        1           
  Lines          73       73           
  Branches       16       16           
=======================================
  Hits           71       71           
  Misses          2        2           
Impacted Files Coverage Δ
src/mongodb-queue.ts 97.26% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 707d3b6...b1675be. Read the comment docs.

@alias-mac alias-mac force-pushed the mongodb-4-compatible branch 2 times, most recently from 3d12a07 to 829489d Compare January 17, 2022 13:46
@alias-mac alias-mac changed the title test: add [email protected] support (testing) feat: add [email protected] support Jan 17, 2022
@alias-mac
Copy link
Contributor Author

Need to wait for mongodb/node-mongodb-native#3102 to land before testing and using the latest version by default.

package.json Outdated
@@ -51,7 +50,7 @@
"husky": "^7.0.4",
"jest": "^27.4.7",
"lint-staged": "^12.1.7",
"mongodb": "^3.6.7",
"mongodb": "~4.1.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will move this to ^4.0.0 once mongodb/node-mongodb-native#3102 is published.

@alias-mac alias-mac force-pushed the mongodb-4-compatible branch 3 times, most recently from 6278973 to f50e7e1 Compare January 19, 2022 09:47
BREAKING CHANGE: it is hard to support both versions (3.x and 4.x) of
native driver on the same branch because 4.x has native types
definitions and they diverge from the ones provided by @types/mongodb.

This means that this version might still be compatible with previous
versions of mongodb native driver (3.6.x and 3.7.x), but we aren't
testing them due to typescript mismatch.

Please open an issue and we will try to support your use case.
@alias-mac alias-mac merged commit 3e37498 into master Jan 19, 2022
@alias-mac alias-mac deleted the mongodb-4-compatible branch January 19, 2022 09:53
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.

1 participant