Skip to content

Keep final PR merges with the user in this repository - #151

Merged
lukemaj merged 1 commit into
mainfrom
fix/override-no-merge
Sep 28, 2026
Merged

lukemaj merged 1 commit into
mainfrom
fix/override-no-merge

Conversation

@lukemaj

@lukemaj lukemaj commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #149 / #150.

AGENTS.override.md granted merging reviewed final PRs, which the user had not agreed to. Now: final PR merges stay with the user; after the user merges, the release workflow and installation on the maintainer's local hosts proceed without another prompt. Test pins the new wording and the absence of the merge grant.

Proof: full repository and versionctl suites OK, versionctl release-check OK (v13.1.1). Independent review follows on this PR.

🤖 Generated with Claude Code

AGENTS.override.md granted merging a reviewed final PR; the user had not
agreed to that. The grant now covers only the release that follows the
user's merge and installation on the maintainer's local hosts.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@lukemaj

lukemaj commented Sep 27, 2026

Copy link
Copy Markdown
Contributor Author

Luna max review of 5627329: success

  1. NON-BLOCKING: tests/test_delivery_continuation_contract.py:129 only rejects the exact old wording, so alternate merge preauthorization could pass. Fix: assert the authority block contains only release and installation operations, with no merge operation.

Validation: targeted tests passed; version mirrors, changelog, and exact base-to-head release checks passed. Full temp-directory tests were unavailable in the read-only sandbox. No files changed or posted.

VERDICT: success

Disposition: non-blocking note left as is; the test pins the new "Final PR merges stay with the user." sentence in addition to rejecting the old wording.

@lukemaj
lukemaj merged commit f49226b into main Sep 28, 2026
3 checks passed
@lukemaj
lukemaj deleted the fix/override-no-merge branch September 28, 2026 14:50
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