Skip to content

fix: admin-merge wfctl update PRs#3

Merged
intel352 merged 1 commit into
mainfrom
fix/wfctl-auto-admin-merge
Jun 7, 2026
Merged

fix: admin-merge wfctl update PRs#3
intel352 merged 1 commit into
mainfrom
fix/wfctl-auto-admin-merge

Conversation

@intel352
Copy link
Copy Markdown
Contributor

@intel352 intel352 commented Jun 7, 2026

Summary

  • use RELEASES_TOKEN when available for wfctl update PR management
  • request Copilot review on generated wfctl update PRs
  • poll generated PR checks and unresolved review threads, then admin-merge once clean

Verification

  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/update-wfctl.yml")'
  • actionlint .github/workflows/update-wfctl.yml
  • git diff --check

Follow-up to homebrew-tap PR #2: repo auto-merge is disabled, so the previous --auto attempt could leave generated formula update PRs open.

Copilot AI review requested due to automatic review settings June 7, 2026 19:50
@intel352 intel352 merged commit e3d55e9 into main Jun 7, 2026
4 checks passed
@intel352 intel352 deleted the fix/wfctl-auto-admin-merge branch June 7, 2026 19:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the update-wfctl GitHub Actions workflow to better manage and finalize automated wfctl formula update pull requests when repo auto-merge is disabled, including using a higher-privilege token when available and performing an admin merge after checks/reviews are clean.

Changes:

  • Use secrets.RELEASES_TOKEN (when present) as GH_TOKEN for PR management operations.
  • Request Copilot review on the generated update PR.
  • Poll PR checks and unresolved review threads, then admin-merge once conditions are clean.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +154 to +159
failed="$(jq '[.[] | select(.bucket == "fail")] | length' <<< "$checks")"
pending="$(jq '[.[] | select(.bucket == "pending")] | length' <<< "$checks")"
if [ "$failed" != "0" ]; then
jq -r '.[] | select(.bucket == "fail") | [.name,.state,.link] | @tsv' <<< "$checks"
exit 1
fi
Comment on lines +179 to +180
gh pr merge "$PR_NUMBER" --squash --admin --delete-branch
exit 0
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.

2 participants