Skip to content

Conversation

@makeitcount
Copy link

This is needed to unblock mongodb node driver upgrade to v6 which has min. version req. of node to be v16. reference

This is needed to unblock mongodb node driver upgrade to v6 which has min. version req. of node to be v16.
[reference](https://github.com/mongodb/node-mongodb-native/blob/HEAD/etc/notes/CHANGES_6.0.0.md)
@makeitcount
Copy link
Author

makeitcount commented Nov 11, 2023

Apart from this more changes will be required in

  • Remove CI tests in sails-mongo that run on node <v16
  • Update sails.js package.json to set node engine requirement to >=16.20.1, remove any test environment with node < v16, change sails version to 2.x to signal breaking change for existing users (we can do this one after sails-mongo PR is merged)

@makeitcount
Copy link
Author

Removed CI tests for v14 and 12 - #2
Added test for v16,18,20

@makeitcount
Copy link
Author

@DominusKelvin please go ahead and merge in your PR if you think this is good

@makeitcount makeitcount changed the title fix: add min node engine requirement to allow es6 features fix: add min node engine requirement to support mandatory es6 requirement by mongodb 6 package Nov 16, 2023
@makeitcount
Copy link
Author

@DominusKelvin did you get a chance to review it?

@DominusKelvin
Copy link
Owner

Hey @makeitcount I am taking a look right now and it does look good. I will revert after the current code has been reviewed

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.

2 participants