Skip to content

chore: update engines requirement in the repo - #768

Merged
gribnoysup merged 1 commit into
mainfrom
update-engines-requirement-npm-11-node-22
Jun 2, 2026
Merged

chore: update engines requirement in the repo#768
gribnoysup merged 1 commit into
mainfrom
update-engines-requirement-npm-11-node-22

Conversation

@gribnoysup

Copy link
Copy Markdown
Collaborator

Preparing for trusted publishing, minimum age, allow scripts, etc.

Comment thread package.json
"email": "compass@mongodb.com"
},
"engines": {
"node": ">=14.17.5",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Tell me if you think there's a reason for us to keep it so low 🤔

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For the top-level package.json? No, doesn't matter

For individual packages? If we think there's realistic external usage, bumping the minimum Node.js version is kind of a breaking change

@gribnoysup gribnoysup Jun 2, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, no, this makes sense, but we only have a top-level engines in the monorepo, none of the packages specify their own. And we don't have versions that are lower in the test matrix, so if there are some requirements there, maybe we should add some for visibility

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage is 78.63%update-engines-requirement-npm-11-node-22 into main. No base build found for main.

@gribnoysup
gribnoysup merged commit af2933f into main Jun 2, 2026
35 checks passed
@gribnoysup
gribnoysup deleted the update-engines-requirement-npm-11-node-22 branch June 2, 2026 08:15
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