Skip to content

chore: update Node.js version in GitHub Actions from 20 to 22#113

Merged
rodrigopavezi merged 1 commit intomainfrom
fix/ci-node-version
Dec 16, 2025
Merged

chore: update Node.js version in GitHub Actions from 20 to 22#113
rodrigopavezi merged 1 commit intomainfrom
fix/ci-node-version

Conversation

@rodrigopavezi
Copy link
Member

@rodrigopavezi rodrigopavezi commented Dec 16, 2025

Problem

Deploy github action is failling as Request Network SDK now requires node version 22: https://github.com/RequestNetwork/payments-subgraph/actions/runs/20264718046/job/58184997535

Changes

Upgrade node version on github action workflow

Summary by CodeRabbit

  • Chores
    • Updated Node.js runtime version in deployment configuration from version 20 to version 22.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 16, 2025

Walkthrough

Updated the Node.js runtime version from 20 to 22 in the GitHub Actions deployment workflow. This changes the Node.js environment used by the setup-node action for subsequent workflow steps.

Changes

Cohort / File(s) Change Summary
GitHub Actions Configuration
\.github/actions/deploy/action.yaml
Updated Node.js version parameter from 20 to 22 in the setup-node action step

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ 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 clearly and concisely summarizes the main change: updating Node.js from version 20 to 22 in GitHub Actions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/ci-node-version

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0657d89 and cf34959.

📒 Files selected for processing (1)
  • .github/actions/deploy/action.yaml (1 hunks)
🔇 Additional comments (1)
.github/actions/deploy/action.yaml (1)

17-17: Node.js 22 update is correct and complete.

The upgrade from Node.js 20 to 22 is verified as compatible with actions/setup-node@v4 and appropriate for the repository. Node.js 22 is now in Active LTS with support through April 30, 2027, while Node.js 20 is in maintenance-only mode. No other workflow files in the repository reference a Node.js version, so no additional updates are needed.


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

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 16, 2025

Greptile Overview

Greptile Summary

Updated Node.js version from 20 to 22 in the deployment action to meet Request Network SDK requirements.

  • Changed node-version from 20 to 22 in .github/actions/deploy/action.yaml
  • This change applies to all deployment jobs (thegraph-studio, core, and mantle) since they all use this shared deploy action
  • No other workflow files require updates as they delegate to this composite action

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is minimal, well-scoped, and directly addresses the deployment failure. The Node.js version bump from 20 to 22 is a straightforward configuration change with no breaking implications for the workflow structure.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
.github/actions/deploy/action.yaml 5/5 Updated Node.js version from 20 to 22 to meet Request Network SDK requirements

Sequence Diagram

sequenceDiagram
    participant GH as GitHub Workflow
    participant Deploy as Deploy Action
    participant Node as Node.js 22
    participant Yarn as Yarn Package Manager
    participant Graph as The Graph CLI
    
    GH->>Deploy: Trigger deployment (push to main)
    Deploy->>Deploy: Checkout code
    Deploy->>Node: Setup Node.js v22
    Node-->>Deploy: Environment ready
    Deploy->>Yarn: Install packages
    Yarn-->>Deploy: Dependencies installed
    Deploy->>Graph: Deploy subgraph to chain
    Graph-->>Deploy: Deployment complete
    Deploy-->>GH: Report success
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link

@bassgeta bassgeta left a comment

Choose a reason for hiding this comment

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

Let's go 💪

@rodrigopavezi rodrigopavezi merged commit b6e3507 into main Dec 16, 2025
5 checks passed
@rodrigopavezi rodrigopavezi deleted the fix/ci-node-version branch December 16, 2025 10:37
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.

3 participants