Skip to content

test(ci): enforce the setup-node cache policy - abandoned - #54

Merged
mldangelo-oai merged 3 commits into
mainfrom
renovate/actions-setup-node-digest
Sep 18, 2026
Merged

mldangelo-oai merged 3 commits into
mainfrom
renovate/actions-setup-node-digest

Conversation

@renovate

@renovate renovate Bot commented Aug 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The original setup-node v6 digest was superseded by #57. This PR keeps the v7 version now on main and adds a regression test for the repository’s existing setup-node cache policy.

  • Check every setup-node invocation in every GitHub workflow, including newly added workflows or jobs.
  • Require the automatic package-manager cache to be explicitly disabled and reject an explicit cache: input, which would otherwise turn it on independently.
  • Keep caching disabled because this Python project invokes the published npm CLI and has no local npm package to cache with setup-node.

Validation

  • 217 unit tests passed locally (10 platform-specific skips on macOS).
  • Exercised two valid false forms and six deliberately unsafe or uncovered workflow cases; all produced the expected result.
  • Ruff lint/format, actionlint, and git diff --check passed.

@renovate
renovate Bot force-pushed the renovate/actions-setup-node-digest branch from 959df74 to 161a2d1 Compare September 18, 2026 18:39
@renovate renovate Bot changed the title chore(deps): update actions/setup-node digest to 2499707 chore(deps): update actions/setup-node digest to 2499707 - autoclosed Sep 18, 2026
@renovate renovate Bot closed this Sep 18, 2026
@renovate
renovate Bot deleted the renovate/actions-setup-node-digest branch September 18, 2026 19:05
@mldangelo-oai mldangelo-oai changed the title chore(deps): update actions/setup-node digest to 2499707 - autoclosed test(ci): enforce the setup-node cache policy Sep 18, 2026
@mldangelo-oai mldangelo-oai reopened this Sep 18, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-18T19:07:48.743050Z 507464f New commits
🔒 Security Review ✅ Completed 2026-09-18T19:07:59.407382Z 507464f New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@renovate renovate Bot changed the title test(ci): enforce the setup-node cache policy test(ci): enforce the setup-node cache policy - abandoned Sep 18, 2026
@renovate

renovate Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Autoclosing Skipped

This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

@mldangelo-oai
mldangelo-oai merged commit 3efd4c8 into main Sep 18, 2026
18 checks passed
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