Skip to content

The standard still describes landing by auto-merge, which is never armed #2272

Open

Description

@fametrano

The maintainer decided on 2026-11-14 that auto-merge is never armed in btclib-org: an armed pull request merges at a moment nobody reads. The merge is an explicit act after the approval, by whoever approves or by a session on his word. PR btclib-org/ai-agents-process#437 writes this into the agents' process.

The standard still describes landing by auto-merge: section 11 of README.md, the verbatim CONTRIBUTING.md (Landing it), and the REPOSITORY.md of several trees. A contributor reads the old rule.

No workflow in any tree arms auto-merge or merges a pull request, and no open pull request is armed (measured on 2026-11-14, recorded in the body of PR btclib-org/ai-agents-process#437).

Done when

  • README.md and the verbatim CONTRIBUTING.md here say the merge is explicit and that auto-merge is not armed, and no passage tells a reader to arm it.
  • The tests that read those passages pass.
  • Each tree's CONTRIBUTING.md copy and REPOSITORY.md are ported.

Whether allow_auto_merge is switched off in the repositories' settings is the maintainer's decision and is not part of this issue.

Activity

  1. fametrano commented on Nov 14, 2026

    @fametrano
    MemberAuthor

    Claimed: a session is writing this in README.md, CONTRIBUTING.md and REPOSITORY.md here, on branch standard-no-auto-merge. The ports to the other trees follow it.

  2. fametrano commented on Nov 14, 2026

    @fametrano
    MemberAuthor

    Hold, for the maintainer's decision (asked 2026-11-14, deferred to the morning report), in two parts. First, whether the pull request for this issue and its ports land through the bypass while the verbatim test is red between the two; the session recommends yes. Second, whether allow_auto_merge is switched off in the repositories' settings now that auto-merge is never armed; the session recommends yes, as a separate change. The branch is written and reviewed meanwhile; it is opened, not landed.

  3. added
    decisionA decision the maintainer still has to take
    on Nov 14, 2026
  4. fametrano commented on Nov 14, 2026

    @fametrano
    MemberAuthor

    The hold above is answered, both parts, by the maintainer on 2026-11-14: the pull request for this issue and its ports land through the bypass once each has the reviews' ACK and no open thread; and allow_auto_merge is switched off in the eighteen repositories, the expected value in this tree's tests changing in the same pull request.

  5. removed
    decisionA decision the maintainer still has to take
    on Nov 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions