fix(ci): repair Dependabot auto-merge workflow - #34
Conversation
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
There was a problem hiding this comment.
Pull request overview
Repairs Dependabot auto-merging while preventing stale workflow runs from merging untested revisions.
Changes:
- Corrects CI-check polling and failure handling.
- Detects Dependabot PRs reliably and cancels stale runs.
- Removes approval and verifies the checked head SHA before merging.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/auto-merge.yml |
Updates Dependabot validation, check polling, concurrency, and merge safeguards. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Balanced
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 139712b26c
ℹ️ 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".
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Summary
gh pr checksJSON fields with the supportedworkflowandbucketfieldsRoot cause
The workflow requested the unsupported
statusandconclusionfields fromgh pr checks. GitHub CLI exited during argument validation, so the polling loop never ran and the merge step was always skipped. After correcting that query, the subsequentgh pr review --approvecommand would also have failed because Actions is not allowed to approve pull requests in this repository.Impact
Dependabot PRs with successful CI can be merged automatically again. Stale runs cannot merge a newer, untested revision of a PR.
Validation
actionlint .github/workflows/*.ymlpassmake buildgo test ./... -count=1go vet ./...golangci-lint run ./...