Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 23, 2025

The renovate bot updated .nvmrc to Node.js 24.11.1 but left workflow files referencing 24.10.0, causing CI failures. Additionally, the update to @types/node 24.x caused type incompatibilities with the CI matrix that tests against Node.js 20.x and 22.x.

Changes

  • Updated node-version matrix in .github/workflows/ci.yml from 24.10.0 to 24.11.1
  • Updated conditional checks (matrix.node-version == '24.10.0') to use 24.11.1 in ci.yml
  • Updated node-version in .github/workflows/coverage.yml to 24.11.1
  • Updated node-version in .github/workflows/release.yml to 24.11.1
  • Downgraded @types/node from 24.10.1 to 20.19.25 to ensure type compatibility across all Node.js versions tested in CI (20.19.0, 22.12.0, and 24.11.1)
  • Regenerated pnpm-lock.yaml to resolve type conflicts caused by mixed @types/node versions

All workflow files now align with the Node.js version specified in .nvmrc, and type checking passes successfully across all tested Node.js versions.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@changeset-bot
Copy link

changeset-bot bot commented Nov 23, 2025

⚠️ No Changeset found

Latest commit: 83b50d9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Nov 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hey-api-docs Ready Ready Preview Comment Nov 24, 2025 2:12am

Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Node.js to version 24.11.1 Fix CI workflows to use Node.js 24.11.1 Nov 23, 2025
Copilot AI requested a review from mrlubos November 23, 2025 22:48
Copilot finished work on behalf of mrlubos November 23, 2025 22:48
@mrlubos mrlubos marked this pull request as ready for review November 24, 2025 00:13
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 24, 2025
@dosubot dosubot bot added lgtm This PR has been approved by a maintainer bug 🔥 Something isn't working labels Nov 24, 2025
@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.01%. Comparing base (88be39d) to head (3b789a5).

Additional details and impacted files
@@                 Coverage Diff                 @@
##           renovate/node-24.x    #3025   +/-   ##
===================================================
  Coverage               33.01%   33.01%           
===================================================
  Files                     426      426           
  Lines                   32792    32792           
  Branches                 2126     2127    +1     
===================================================
  Hits                    10827    10827           
  Misses                  21937    21937           
  Partials                   28       28           
Flag Coverage Δ
unittests 33.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mrlubos
Copy link
Member

mrlubos commented Nov 24, 2025

@copilot CI still failing

The issue was that @types/node 24.x is incompatible with Node.js 20.x and 22.x
in the CI matrix. Using @types/node 20.x ensures compatibility across all tested
Node.js versions (20.19.0, 22.12.0, and 24.11.1).

Co-authored-by: mrlubos <12529395+mrlubos@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 24, 2025

@copilot CI still failing

Fixed! The CI was failing due to type incompatibilities caused by @types/node 24.x being incompatible with Node.js 20.x and 22.x. I've downgraded @types/node to v20.19.25 to ensure compatibility across all Node.js versions in the CI matrix (20.19.0, 22.12.0, and 24.11.1), and regenerated the pnpm-lock.yaml to resolve the conflicts. (commit 83b50d9)

Copilot AI changed the title Fix CI workflows to use Node.js 24.11.1 Fix CI workflows to use Node.js 24.11.1 and resolve type compatibility issues Nov 24, 2025
Copilot AI requested a review from mrlubos November 24, 2025 02:15
Copilot finished work on behalf of mrlubos November 24, 2025 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Something isn't working lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants