Skip to content

Consolidate auto-merge and mark PR commit status - #6

Merged
netbirddev merged 4 commits into
netbirddev:mainfrom
mlsmaycon:auto-merge-consolidate
Aug 22, 2026
Merged

Consolidate auto-merge and mark PR commit status#6
netbirddev merged 4 commits into
netbirddev:mainfrom
mlsmaycon:auto-merge-consolidate

Conversation

@mlsmaycon

Copy link
Copy Markdown
Collaborator
  • Remove the same-repo-only auto-merge job from ci.yml; Fork Auto Merge (workflow_run) is now the single merge path for both fork and maintainer PRs.
  • Fork Auto Merge runs a privileged test job and publishes a commit status on the PR head SHA, so the workflow_run result is linked to the PR and tracked with the other workflows.
  • Fork Test now records metadata first and uploads it with if: always() so a status can be posted even when tests fail.

Claude-Session: https://claude.ai/code/session_01GHWDrXhQK3GvjLBjuHsZth

claude and others added 3 commits August 22, 2026 13:31
- Remove the same-repo-only auto-merge job from ci.yml; Fork Auto Merge
  (workflow_run) is now the single merge path for both fork and
  maintainer PRs.
- Fork Auto Merge runs a privileged test job and publishes a commit
  status on the PR head SHA, so the workflow_run result is linked to the
  PR and tracked with the other workflows.
- Fork Test now records metadata first and uploads it with if: always()
  so a status can be posted even when tests fail.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Claude-Session: https://claude.ai/code/session_01GHWDrXhQK3GvjLBjuHsZth
An immediate gh pr merge fails with 'base branch policy prohibits the
merge' when main has required checks/reviews still pending. --auto queues
the PR and lets GitHub merge once requirements are met (and re-checks the
current head, covering the TOCTOU concern). Requires 'Allow auto-merge'
enabled in repo settings.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Claude-Session: https://claude.ai/code/session_01GHWDrXhQK3GvjLBjuHsZth
@netbirddev
netbirddev merged commit 2701d48 into netbirddev:main Aug 22, 2026
3 checks passed
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.

3 participants