Skip to content

chore: replaces volta with actions/setup-node#2644

Merged
stalniy merged 1 commit intomainfrom
chore/gh-nodejs
Feb 3, 2026
Merged

chore: replaces volta with actions/setup-node#2644
stalniy merged 1 commit intomainfrom
chore/gh-nodejs

Conversation

@stalniy
Copy link
Contributor

@stalniy stalniy commented Feb 2, 2026

Why

To rely on more standard actions, instead on custom ones. Better for security

Summary by CodeRabbit

  • Chores
    • Enhanced CI/CD pipeline reliability by updating Node.js setup mechanism in build workflows
    • Improved build validation checks with strengthened retry logic to better handle intermittent failures

@stalniy stalniy requested a review from a team as a code owner February 2, 2026 09:29
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

GitHub Actions workflows are updated across three files: two workflows migrate Node.js setup from volta-cli/action to actions/setup-node, while a third workflow adjusts retry parameters (3→5 retries, 15→30 second timeout) and simplifies status validation logic in codecov patch checks.

Changes

Cohort / File(s) Summary
Node.js Setup Migration
.github/workflows/create-pre-release-pr.yml, .github/workflows/generate-claude-md.yml
Replaces volta-cli/action@v4 with actions/setup-node@v6, adds node-version-file pointing to package.json to derive Node.js version from package configuration.
Codecov Retry Logic
.github/workflows/reusable-validate-app.yml
Increases retry count from 3 to 5 and sleep timeout from 15 to 30 seconds. Simplifies status check by removing explicit empty/null handling and consolidating all non-"completed" statuses under uniform retry logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • ygrishajev
  • baktun14

Poem

🐰 Node.js setup hops with newer grace,
Actions replace volta's embrace,
Retry patience grows, timeouts extend,
Status checks simplified, logic trends,
Workflows flutter, workflows flow! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: replaces volta with actions/setup-node' accurately describes the main change across all modified workflow files, which consistently replace volta-cli/action@v4 with actions/setup-node@v6.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/gh-nodejs

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Feb 2, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
2154 1 2153 1
View the top 1 failed test(s) by shortest run time
Addresses API GET /v1/addresses/{address}/deployments/{skip}/{limit} returns 400 when skip is not a number
Stack Traces | 0.079s run time
Error: 
    at Query.run (.../dialects/postgres/query.js:76:25)
    at .../sequelize/src/sequelize.js:650:28
    at processTicksAndRejections (node:internal/process/task_queues:103:5)
    at PostgresQueryInterface.insert (.../dialects/abstract/query-interface.js:795:21)
    at Lease.save (.../sequelize/src/model.js:4154:35)
    at lease.create (.../sequelize/src/model.js:2305:12)
    at createLease (.../test/seeders/lease.seeder.ts:6:10)
    at async Promise.all (index 1)
    at setup (.../test/functional/addresses.spec.ts:356:5)
    at Object.<anonymous> (.../test/functional/addresses.spec.ts:190:27)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@stalniy stalniy merged commit be3af05 into main Feb 3, 2026
73 of 75 checks passed
@stalniy stalniy deleted the chore/gh-nodejs branch February 3, 2026 04:50
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.

2 participants

Comments