chore(deps): update GitHub Actions deps, upgrade Node.js to v20, remove outdated overrides#32
chore(deps): update GitHub Actions deps, upgrade Node.js to v20, remove outdated overrides#32alexslavr wants to merge 9 commits intoDevExpress:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates this monorepo’s GitHub Action tooling to newer dependency versions and Node 20, aligning action runtime metadata with the repository’s Node 20 baseline while cleaning up no-longer-needed pnpm overrides.
Changes:
- Switch action runtimes from
node16tonode20in action metadata. - Bump
@actions/*dependencies via the pnpm workspace catalog and refreshpnpm-lock.yaml. - Remove outdated pnpm
overridesentries.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pr-filter/action.yml | Updates the action runtime to node20. |
| get-changed-files/action.yml | Updates the action runtime to node20. |
| pnpm-workspace.yaml | Bumps @actions/* catalog versions and removes outdated overrides. |
| pnpm-lock.yaml | Lockfile refresh reflecting the catalog bumps and override removals. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| '@actions/core': 2.0.3 | ||
| '@actions/exec': 2.0.0 | ||
| '@actions/github': 8.0.1 |
There was a problem hiding this comment.
Pull request overview
Upgrades this monorepo of JavaScript GitHub Actions to Node.js 20 and refreshes the shared dependency catalog/lockfile to newer @actions/* and Octokit versions, while dropping no-longer-needed override entries.
Changes:
- Bump Node engine requirements to
>=20.0.0across action packages (and update affected action runtimes tonode20). - Update workspace catalog versions for
@actions/*and@types/nodeand regeneratepnpm-lock.yamlaccordingly. - Remove outdated
pnpmoverrides that are no longer needed with the updated dependency graph.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| verify-version-change/package.json | Raises Node engine requirement to Node 20. |
| tsconfig.base.json | Updates TS target/lib baseline to ES2022. |
| send-teams-notification/package.json | Raises Node engine requirement to Node 20. |
| pr-filter/tsconfig.json | Adds explicit Node typings for TS compilation. |
| pr-filter/package.json | Raises Node engine requirement and adds @types/node. |
| pr-filter/action.yml | Switches GitHub Action runtime from node16 to node20. |
| pnpm-workspace.yaml | Updates dependency catalog versions; removes outdated overrides. |
| pnpm-lock.yaml | Reflects updated catalog versions and transitive dependency resolution. |
| get-changed-files/package.json | Raises Node engine requirement to Node 20. |
| get-changed-files/action.yml | Switches GitHub Action runtime from node16 to node20. |
| common/package.json | Raises Node engine requirement to Node 20. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| '@actions/core': 2.0.3 | ||
| '@actions/exec': 2.0.0 | ||
| '@actions/github': 8.0.1 | ||
| '@types/jest': ^29.5.12 | ||
| '@types/node': ^16.18.105 | ||
| '@types/node': ^20.19.37 |
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s GitHub Action toolchain to align with Node.js 20, refreshes shared dependency versions, and removes no-longer-needed pnpm overrides to simplify maintenance and reduce dependency risk.
Changes:
- Bump Node.js engine requirements to
>=20.0.0across workspace packages. - Update GitHub Actions toolkit dependencies (e.g.,
@actions/*,@actions/github→ newer major versions) and refresh the lockfile accordingly. - Modernize TypeScript baseline (
target/lib→ ES2022) and adjust per-package TS typing where needed.
Reviewed changes
Copilot reviewed 10 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| verify-version-change/package.json | Align action package Node engine requirement with Node 20. |
| tsconfig.base.json | Move shared TS compilation baseline to ES2022 for Node 20-era runtimes. |
| send-teams-notification/package.json | Align action package Node engine requirement with Node 20. |
| pr-filter/tsconfig.json | Ensure Node typings are explicitly included for TS compilation. |
| pr-filter/package.json | Align Node engine requirement and add @types/node to match TS config. |
| pr-filter/action.yml | Switch GitHub Action runtime from node16 to node20. |
| pnpm-workspace.yaml | Update catalog dependency versions and remove outdated overrides. |
| pnpm-lock.yaml | Reflect dependency upgrades/removals and updated override set in a consistent lockfile. |
| get-changed-files/package.json | Align action package Node engine requirement with Node 20. |
| get-changed-files/action.yml | Switch GitHub Action runtime from node16 to node20. |
| common/package.json | Align shared library package Node engine requirement with Node 20. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "compilerOptions": { | ||
| "target": "ES6", | ||
| "target": "ES2022", | ||
| "lib": ["ES2022"], |
There was a problem hiding this comment.
Is lib necessary?
I think lib is used to specify extra typings not included into target (like when using polyfills)
BREAKING CHANGE: upgrade Node.js to v20 for get-changed-files and pr-filter actions