Repository navigation
ci: move every ci-workflows pin to v0.50.0 - #481
Conversation
v0.50.0 adds npm, pip and NuGet patch and minor auto-merge (melodic-software/ci-workflows#783); majors still wait for review. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-review reviewed Incremental review of The change moves each
I did not run the Node pin tests, actionlint or |
|
claude-security-review reviewed Security review (incremental, 8b3e08c..d0d8ecd): no findings. The change since the last review is a pin bump of
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b3e08c9fb
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Product policy requires every canonical ci-workflows reference to share one pin, and release/dependencies.json to name it. The previous commit moved only pr-automerge-dependabot.yml. Moved from 594987abe8657a40bccc13d301f1f0d5cccb5348 # v0.39.8 to d223a6446680fb261e264d1dea9efba46f20ead2 # v0.50.0: - .github/workflows/pr-require-checks.yml: all 19 composite and reusable references (lint actions, pr-audit-workflows, pr-run-checks-go, pr-require-checks/check-contract and aggregate-results) - release/dependencies.json: repositoryPins commit and source No caller changes. Between the two releases, check-contract gained an optional issue-ref-pattern input whose default is the old GitHub grammar, pr-audit-workflows only bumped codeql-action upload-sarif, and pr-automerge-dependabot gained optional package-auto-merge and review-only-dependencies inputs; its package path covers npm, pip and NuGet, none of which this repository's Dependabot config updates. The SYNC-MANAGED callers stay on their standards-sync pins. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No related issue: org rollout of package auto-merge after melodic-software/ci-workflows#783
Summary
Moves every canonical ci-workflows reference in this repository to v0.50.0, which adds npm, pip and NuGet patch and minor auto-merge to the shared Dependabot caller. Product policy requires all canonical references to share one pin, so the auto-merge caller cannot move alone.
Fix
@594987abe8657a40bccc13d301f1f0d5cccb5348 # v0.39.8becomes@d223a6446680fb261e264d1dea9efba46f20ead2 # v0.50.0in:.github/workflows/pr-automerge-dependabot.yml(1 reference).github/workflows/pr-require-checks.yml(all 19 composite and reusable references)release/dependencies.json(repositoryPinscommit and source)No caller changes. Between the two releases,
pr-require-checks/check-contractgained an optionalissue-ref-patterninput whose default is the old GitHub grammar;pr-audit-workflows.ymlonly bumpedcodeql-action/upload-sarif;pr-automerge-dependabot.ymlgained optionalpackage-auto-merge(default true) andreview-only-dependenciesinputs. Its package path covers npm, pip and NuGet; this repository's Dependabot config updates only gomod, docker and github-actions, so it changes nothing here. The other changed actions and workflows are not called by this repository. The SYNC-MANAGED callers (pr-check-managed-files-hosted.yml,pr-review-hosted.yml,pr-review-security-hosted.yml) stay on their standards-sync pins.Verification
node --test .github/scripts/*.test.cjs: 23 pass, 0 fail, including the three pin tests that failed before.actionlinton both changed workflows: clean.scripts/Test-ReleasePins.ps1: "Release and workflow pins are internally consistent."git grepfinds no remaining594987aorv0.39.8.Related
🤖 Generated with Claude Code