Skip to content

Pin @types/node to v24 - #1585

Merged
v-HarithaVattikuti merged 1 commit into
actions:mainfrom
v-priyagupta108:pin-types-node-v24
Jul 16, 2026
Merged

Pin @types/node to v24#1585
v-HarithaVattikuti merged 1 commit into
actions:mainfrom
v-priyagupta108:pin-types-node-v24

Conversation

@v-priyagupta108

Copy link
Copy Markdown
Contributor

Description:
Pin @types/node to ^24.1.0 to match the action's supported runtime (engines.node >=24.0.0) and keep type definitions in sync.

Related issue:
Add link to the related issue.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

CopilotAI review requested due to automatic review settings July 16, 2026 09:36
@v-priyagupta108
v-priyagupta108 requested a review from a team as a code ownerJuly 16, 2026 09:36
@v-priyagupta108v-priyagupta108 self-assigned this Jul 16, 2026

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates this action’s TypeScript type definitions to align with the supported Node.js runtime (engines.node >=24.0.0) by pinning @types/node to the v24 major line.

Changes:

  • Change @types/node devDependency from ^26.0.0 to ^24.1.0.
  • Regenerate package-lock.json to reflect the new @types/node resolution (including undici-types resolution updates and removal of now-unneeded optional peer installs).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

FileDescription
package.jsonPins @types/node to the v24 major line to match engines.node.
package-lock.jsonUpdates lockfile resolution to @types/node@24.x and associated dependency graph changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@v-HarithaVattikuti
v-HarithaVattikuti merged commit 05e62e6 into actions:mainJul 16, 2026
231 checks passed
Sign up for freeto 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.

5 participants

@v-priyagupta108@v-HarithaVattikuti@v-priya-kinthali@v-gowridurgad