Skip to content

ci: auto-merge this repo's own Dependabot PRs - #197

Merged
twistedmelonman merged 2 commits into
mainfrom
claude/feat-self-dependabot-auto-merge-055823
Oct 19, 2026
Merged

twistedmelonman merged 2 commits into
mainfrom
claude/feat-self-dependabot-auto-merge-055823

Conversation

@twistedmelonman

Copy link
Copy Markdown
Member

What was broken

This repo defines dependabot-auto-merge.yml but never called it, so every Dependabot PR here waited for a hand merge (most recently #196).

How it is fixed

  • New self-dependabot-auto-merge.yml calls the reusable workflow pinned at @dependabot-auto-merge-v2, not a local path. pull_request_target runs the base branch's copy, so a local path would let an unreleased change decide merges here.
  • Safe because every fleet caller pins a hand-moved tag: a merge to main here reaches no other repo. claude-config and dotfiles, also merge-locked, already auto-merge their Dependabot PRs.
  • guard-no-checkout now checks the new caller too, since it also runs on pull_request_target.
  • README notes the new caller.

Verification

  • actionlint, yamllint, zizmor clean on both workflow files; markdownlint clean with standards/markdownlint.json.
  • Ran the guard step's script (extracted with yq) against the real files (pass) and a copy with uses: actions/checkout@v4 appended to the caller (fails, exit 1).

Claude Code Bot and others added 2 commits October 19, 2026 16:08
github-workflows defines dependabot-auto-merge.yml but never called it,
so every Dependabot PR here waited for a hand merge (most recently #196).
Every fleet caller pins a hand-moved tag, so a merge to main here reaches
no other repo, and the other locked repos (claude-config, dotfiles)
already auto-merge their Dependabot PRs.

The caller pins @dependabot-auto-merge-v2 instead of a local path:
pull_request_target runs the base branch's copy, so a local path would
let an unreleased change to the reusable workflow decide merges here.

guard-no-checkout now checks the new caller too, since it also runs on
pull_request_target. Checked against a copy with a checkout step added:
the guard fails it and passes the real files.
@twistedmelonman
twistedmelonman merged commit d98d58e into main Oct 19, 2026
2 checks passed
@twistedmelonman
twistedmelonman deleted the claude/feat-self-dependabot-auto-merge-055823 branch October 19, 2026 23:19
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