Skip to content

Conversation

@iodic
Copy link
Collaborator

@iodic iodic commented Feb 27, 2025

Changes proposed in this Pull Request:

This changeset includes the following changes:

  • Updates the NPM dependencies to latest version (the @wordpress packages).
  • Updates .nvmrc to use the latest LTS node version.
  • Removes the node & NPM engines declaration in package.json — we want to aim to have the packages up to date & secure.

Detailed test instructions:

As this affects the build process, we need to do end to end testing to ensure these updates do not introduce regressions.

"use_gh_release_notes": true
},
"engines": {
"node": ">=12.22 <=16",
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we need to set specific versions of npm/node that we want to test this on?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are right; I have set them to a minimum of the latest node LTS & matching npm version.

@iodic iodic force-pushed the chore/update-node-packages branch 2 times, most recently from 14e30c8 to c97914e Compare March 4, 2025 09:51
@vinkmeta
Copy link
Contributor

@iodic What is the plan for this PR ? Can we merge this by next week ?

@iodic iodic force-pushed the chore/update-node-packages branch from c97914e to f98e0b7 Compare March 19, 2025 09:30
@iodic iodic force-pushed the chore/update-node-packages branch from f98e0b7 to 6e249cd Compare March 31, 2025 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants