Skip to content

agents: explain force-merging while requiring user consent - #3998

Merged
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:expose_worker_stderr_logs
Aug 4, 2026
Merged

agents: explain force-merging while requiring user consent#3998
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:expose_worker_stderr_logs

Conversation

@rickeylev

@rickeylev rickeylev commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

The merge-pr skill tells agents how to force merge pull requests using
gh pr merge --admin, but also explicitly requires that agents obtain
user consent before bypassing merge queues or required checks.

Update the pull request rules to explicitly prohibit per-file edit
lists or individual file changelog items in PR descriptions and commit
messages. Reinforce that PR descriptions must provide a high-level
overview explaining why a change is made and how at a conceptual
level, and mandate linking related issues.
Add instructions to the merge-pr skill for handling explicit force
merge requests using the gh pr merge --admin flag to bypass the merge
queue. Emphasize that passing --admin strictly requires explicit
user permission or consent and must never be invoked autonomously.
@rickeylev
rickeylev requested a review from aignas as a code owner August 4, 2026 02:54
@rickeylev rickeylev changed the title agents(merge-pr): explain force-merging while requiring user consent agents: tell agents to not include per-file changes and explain force-merging while requiring user consent Aug 4, 2026
@rickeylev rickeylev changed the title agents: tell agents to not include per-file changes and explain force-merging while requiring user consent agents: explain force-merging while requiring user consent Aug 4, 2026
@rickeylev
rickeylev merged commit 874a683 into bazel-contrib:main Aug 4, 2026
8 of 10 checks passed
@rickeylev
rickeylev deleted the expose_worker_stderr_logs branch August 4, 2026 02:56
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