Skip to content

chore: update Actions version - #4512

Open
ysknsid25 wants to merge 1 commit into
nitrojs:mainfrom
ysknsid25:chore/update-actions-version
Open

chore: update Actions version#4512
ysknsid25 wants to merge 1 commit into
nitrojs:mainfrom
ysknsid25:chore/update-actions-version

Conversation

@ysknsid25

Copy link
Copy Markdown
Contributor

🔗 Linked issue

resolves: #4511

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

Signed-off-by: ysknsid25 <kengo071225@gmail.com>
@ysknsid25
ysknsid25 requested a review from pi0 as a code owner August 8, 2026 07:12
@vercel

vercel Bot commented Aug 8, 2026

Copy link
Copy Markdown

@ysknsid25 is attempting to deploy a commit to the Nitro Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The pull request updates pinned actions/checkout and actions/setup-node versions across four workflow files. It also updates denoland/setup-deno in the Rollup and Rolldown test jobs. Other workflow behavior remains unchanged.

Changes

Workflow action upgrades

Layer / File(s) Summary
Workflow action pin updates
.github/workflows/*.yml
The workflows use actions/checkout v7.0.1 and actions/setup-node v7.0.0. The Rollup and Rolldown jobs use denoland/setup-deno v2.0.5.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • nitrojs/nitro#4278: Updates GitHub Actions references in overlapping workflow files with additional SHA pinning and workflow changes.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the conventional commit format and clearly describes the workflow action version updates.
Description check ✅ Passed The description identifies the linked issue and categorizes the change as a chore for updating GitHub Actions versions.
Linked Issues check ✅ Passed The changes update checkout, setup-node, and setup-deno versions in all workflows listed by issue #4511.
Out of Scope Changes check ✅ Passed All changes are limited to the workflow action version updates required by issue #4511.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@pkg-pr-new

pkg-pr-new Bot commented Aug 8, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/nitro@4512

commit: 74d2061

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/npm-publish.yml:
- Around line 29-32: Remove the cache: pnpm setting from the actions/setup-node
step in the npm publish workflow, while preserving node-version: lts/* and the
existing tag-trigger configuration.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7f9f72e6-63dd-43c1-ba7b-c73d67bc8d26

📥 Commits

Reviewing files that changed from the base of the PR and between 52abde8 and 74d2061.

📒 Files selected for processing (4)
  • .github/workflows/autofix.yml
  • .github/workflows/ci.yml
  • .github/workflows/copilot-setup-steps.yml
  • .github/workflows/npm-publish.yml

Comment thread .github/workflows/npm-publish.yml
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.

chore: Update Actions Version

1 participant