Skip to content

Revert "Bump actions/setup-node from 4 to 5"#3660

Merged
acicovic merged 1 commit intodevelopfrom
revert-3659-dependabot/github_actions/actions/setup-node-5
Sep 8, 2025
Merged

Revert "Bump actions/setup-node from 4 to 5"#3660
acicovic merged 1 commit intodevelopfrom
revert-3659-dependabot/github_actions/actions/setup-node-5

Conversation

@acicovic
Copy link
Collaborator

@acicovic acicovic commented Sep 8, 2025

Reverts #3659, to see whether it's related to a build issue we're experiencing in the develop branch.

Summary by CodeRabbit

  • Chores
    • Updated continuous integration workflows to use a different Node.js setup action across build, end-to-end tests, and Node pipelines.
    • Build steps and test execution remain unchanged; this aligns the environment configuration without affecting functionality.
    • No user-facing changes; application behavior and features are unaffected.

@acicovic acicovic added this to the 3.20.7 milestone Sep 8, 2025
@acicovic acicovic self-assigned this Sep 8, 2025
@acicovic acicovic requested a review from a team as a code owner September 8, 2025 07:07
@acicovic acicovic added Deps: GitHub Actions Reserved to dependency/dependabot PRs Deps Reserved to dependency/dependabot PRs labels Sep 8, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 8, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updates three GitHub Actions workflows to use actions/setup-node v4/v4.1.0 instead of v5 in their Node.js setup steps; all other steps and inputs remain unchanged.

Changes

Cohort / File(s) Summary
GitHub Actions workflows: setup-node version pin
\.github/workflows/build-plugin.yml, \.github/workflows/e2e-tests.yml, \.github/workflows/node.js.yml
Downgrade Node.js setup step from actions/setup-node@v5 to actions/setup-node@v4 or @v4.1.0; inputs (node-version, cache) and surrounding steps unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Maintenance


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 494dbea and 75d1453.

📒 Files selected for processing (3)
  • .github/workflows/build-plugin.yml (1 hunks)
  • .github/workflows/e2e-tests.yml (1 hunks)
  • .github/workflows/node.js.yml (1 hunks)
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-3659-dependabot/github_actions/actions/setup-node-5

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 and usage tips.

@acicovic acicovic merged commit ca28834 into develop Sep 8, 2025
33 checks passed
@acicovic acicovic deleted the revert-3659-dependabot/github_actions/actions/setup-node-5 branch September 8, 2025 07:12
@acicovic acicovic removed Deps Reserved to dependency/dependabot PRs Deps: GitHub Actions Reserved to dependency/dependabot PRs labels Sep 8, 2025
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.

1 participant