-
Notifications
You must be signed in to change notification settings - Fork 237
ci: update environment #668
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: 5.x
Are you sure you want to change the base?
Conversation
30a3f10
to
71e86b9
Compare
- Test against MongoDB v3.6 and v5.0 - fix Travis CI config - Use `npm ci --ignore-scripts` Signed-off-by: Rifa Achrinza <25147899+achrinza@users.noreply.github.com>
Signed-off-by: Rifa Achrinza <25147899+achrinza@users.noreply.github.com>
22b1eab
to
0659b3e
Compare
Signed-off-by: Rifa Achrinza <25147899+achrinza@users.noreply.github.com>
0659b3e
to
701da2e
Compare
@@ -16,7 +16,8 @@ jobs: | |||
strategy: | |||
matrix: | |||
node-version: [10, 12, 14] | |||
mongodb-version: [4.4] # the latest stable version | |||
mongodb-version: [3.6, 4.4, 5.0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
according to https://www.mongodb.com/support-policy/lifecycles, mongodb v3.6 has reached EOL last April. Do we still want to include that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since connector 5.x was released when MongoDB 3.x was still supported, I wanted to make sure it still works with that version (i.e. Dropping a MongoDB version should require a semver major on the connector as well).
Signed-off-by: Rifa Achrinza 25147899+achrinza@users.noreply.github.com
Checklist
npm test
passes on your machine