Skip to content

ci(config-ui): bump actions/setup-node from 6 to 7 - #9308

Open
DoDiODev wants to merge 1 commit into
apache:mainfrom
DoDiODev:pr/setup-node-v7
Open

DoDiODev wants to merge 1 commit into
apache:mainfrom
DoDiODev:pr/setup-node-v7

Conversation

@DoDiODev

@DoDiODev DoDiODev commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Takes over the Dependabot bump in #9119 (actions/setup-node 6 → 7), which
has been open since September and is 114 commits behind main.

  • .github/workflows/config-ui.yml: actions/setup-node@v6 → @v7. This is
    the only use of setup-node in the repo.
  • Same file: add .github/workflows/config-ui.yml to the paths filter.
    So far the workflow only ran for config-ui/** changes, so a PR that edits
    the workflow itself never ran it. build(deps): bump actions/setup-node from 6 to 7 #9119 has 14 green checks, but none of them
    executed setup-node@v7. With this change the config-ui job runs on this
    PR and on future action bumps.

v7 is a major release (ESM migration, @actions/cache 5.1.0, no dummy
NODE_AUTH_TOKEN export). We use none of the affected inputs (registry-url,
always-auth, mirrors), only node-version: '24' and the yarn cache.

Does this close any open issues?

Supersedes #9119.

Screenshots

N/A

Other Information

Verified on the fork before opening this PR:
https://github.com/DoDiODev/devlake/actions/runs/41451353013. All jobs passed,
including config-ui (yarn tsc && yarn lint). The log shows
Download action repository 'actions/setup-node@v7', Node 24.22.1 and a yarn
cache hit. actions/* is implicitly allowed by the ASF action allowlist, and
the ASF action allowlist check passed as well.

Supersedes the Dependabot PR apache#9119. setup-node v7 is a major release
(ESM migration, dependency upgrades, no dummy NODE_AUTH_TOKEN export).

Also trigger the config-ui workflow when the workflow file itself
changes. Before, its `paths: config-ui/**` filter meant that a change to
config-ui.yml (like this bump) never ran the job it modifies, so the
action upgrade went untested in CI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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