Repository navigation
build: bump github/codeql-action/upload-sarif from 4.38.1 to 4.38.2 in the vendor-actions group across 1 directory - #674
Merged
kyle-sexton merged 1 commit intoOct 5, 2026
Conversation
Bumps the vendor-actions group with 1 update in the / directory: [github/codeql-action/upload-sarif](https://github.com/github/codeql-action). Updates `github/codeql-action/upload-sarif` from 4.38.1 to 4.38.2 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@1c5b675...2892aa5) --- updated-dependencies: - dependency-name: github/codeql-action/upload-sarif dependency-version: 4.38.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: vendor-actions ... Signed-off-by: dependabot[bot] <support@github.com>
kyle-sexton
added a commit
that referenced
this pull request
Oct 4, 2026
No related issue: Phase 5 plan step (shared Dependabot auto-merge reusable R); tracked in the actions-conventions plan, not an issue. ## Summary Adds `.github/workflows/pr-automerge-dependabot.yml`, a `workflow_call` reusable that arms GitHub squash auto-merge on low-risk Dependabot PRs. GitHub then merges once `ci-status` and every other required check pass. It implements standards `components/dependabot-policy/policy.json` `autoMerge` (`github-actions`; `actions/*`, `github/*`, `anthropics/*`; no semver-major; `ci-status`). ## Fix - Gates, all required: event is `pull_request`; PR author id `49699333` (dependabot[bot]); event sender (`github.event.sender.id`, the pusher) is `49699333`; fetch-metadata succeeded, and failed, empty or invalid metadata is a skip; every update's ecosystem is `github_actions`; every dependency name matches the case-sensitive allowlist glob; every update is semver patch or minor (grouped PRs included; an uncomputable type skips); every PR commit, head included, has author `49699333`, committer `19864447` (web-flow) and a signature verified with reason `valid`. That commit shape matches live Dependabot commits on medley#2094 and ci-workflows #651, #655 and #674. - fetch-metadata is `continue-on-error`, and the gate step runs on `!cancelled()`. A force-push that makes fetch-metadata fail therefore still reaches the disarm. - Arms with `enablePullRequestAutoMerge(mergeMethod: SQUASH, expectedHeadOid: <verified head>)`. No direct-merge fallback, so nothing merges around `ci-status`. - A skipped PR that was armed on an earlier head is disarmed (GitHub keeps auto-merge on after a push by a write-access user). - No checkout; PR-derived values reach the script via `env:` only. Job permissions: `contents: write`, `pull-requests: write`. Input: `runner` (default `ubuntu-24.04`). - The gate runs inline in `actions/github-script` because a reusable has no ci-workflows checkout. `.github/scripts/pr-automerge-dependabot.test.cjs` extracts the script and runs it against fixtures, picked up by `test-contracts` (`node --test .github/scripts/*.test.cjs`). - README entry with the canonical caller. ## Verification - `node --test .github/scripts/*.test.cjs`: 219 pass, 0 fail (27 in the new file: vendor patch armed, vendor major, community patch, mixed group, vendor group with one major, non-Dependabot author, non-Dependabot sender, foreign head commit, forged author with a non-web-flow committer, unsigned commit, non-`valid` reason, moved head, non-actions ecosystem, lookalike owner, `Actions/checkout` case variant, empty and invalid metadata, failed metadata fetch disarmed, disarm). - Mutation spot-checks (unanchored glob, head-moved check, disarm, signature, committer, reason, sender, metadata outcome, continue-on-error) each turn a test red. - actionlint 1.7.12 and zizmor 1.30.0 clean on the new workflow; markdownlint and typos clean. - naming-lint `--mode enforcing` exits 0 after merging origin/main (#675). ## Related - Phase 5 plan Phase 3 (R). Next: Fable review, a minor release, then the standards `approvedReusableWorkflowContracts` entry and the allowlist lockstep test (separate PRs). - Ports medley `.github/workflows/dependabot-automerge.yml`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
kyle-sexton
deleted the
dependabot/github_actions/vendor-actions-b300d6d0aa
branch
October 5, 2026 00:29
kyle-sexton
added a commit
that referenced
this pull request
Oct 5, 2026
No related issue: found by the actions-conventions Phase 8 sanity check (16 of 17 repos passed; ci-workflows lacked an auto-merge caller). ## Summary ci-workflows is the only repository without a caller of the shared `pr-automerge-dependabot` reusable, so its own vendor Dependabot bumps (e.g. #674) never arm auto-merge, contrary to the org Dependabot lanes policy (standards#684). A separate caller file cannot exist here because `pr-automerge-dependabot.yml` is the reusable itself, so this adds a thin dogfood caller job to `pr-require-checks.yml`, the pattern the Claude lanes already use. ## Fix - New job `pr-automerge-dependabot` in `.github/workflows/pr-require-checks.yml`: `pull_request` events only, the same action predicate as `pr-review`. It has job-scoped `contents: write` and `pull-requests: write`; workflow-level permissions are unchanged. - It is pinned to the released reusable (`@fb56986… # v0.36.0`), not `./`, so a pull request, a Dependabot bump of an action inside the reusable included, cannot redefine the code that holds the write token. - `ci-status` does not depend on the job, because arming waits on `ci-status`. `ci-fanout-consolidation.test.cjs` now lists it among the jobs that stay out of `ci-status`. ## Verification - `actionlint .github/workflows/pr-require-checks.yml`: exit 0. - `node --test .github/scripts/*.test.cjs`: all pass. - The Claude review lanes skip pull requests that edit their caller workflow (`pr-require-checks.yml`); a local review stands in for them. - To confirm after merge: the next vendor-actions Dependabot PR in this repo shows a passing `pr-automerge-dependabot / …` check and auto-merge armed. ## Related - standards#684 (Dependabot lanes policy) - #674 (vendor bump merged by hand for lack of this caller) - #676 (the reusable) 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the vendor-actions group with 1 update in the / directory: github/codeql-action/upload-sarif.
Updates
github/codeql-action/upload-sariffrom 4.38.1 to 4.38.2Release notes
Sourced from github/codeql-action/upload-sarif's releases.
Changelog
Sourced from github/codeql-action/upload-sarif's changelog.
... (truncated)
Commits
2892aa5Merge pull request #4168 from github/update-v4.38.2-a6ef2c96f8ad03a3Trigger workflows98af865Update changelog for v4.38.2a6ef2c9Merge pull request #4156 from github/mario-campos/fix-validate-cmd1ef28a1Merge pull request #4166 from github/dependabot/github_actions/dot-github/wor...26cb08bMerge pull request #4163 from github/mbg/fix-getCommitOid-stubsf035ce3Merge pull request #4165 from github/dependabot/npm_and_yarn/npm-minor-8eaed9...5e4e255Rebuildb13f5f4Bump ruby/setup-rubyc87fe57RebuildDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions