From 59ad04e740577300fb472be2028825fc94ee8982 Mon Sep 17 00:00:00 2001 From: Kyle Sexton <153232337+kyle-sexton@users.noreply.github.com> Date: Sat, 17 Oct 2026 22:17:37 -0400 Subject: [PATCH 1/2] docs(pr-pipeline): hold a lane PR on a not-verified item that can stop the lane ci-workflows#755 shipped an intake author gate it listed as "unverified, fails closed"; it failed closed and skipped every intake run. A lane PR with such an item now waits for a live check, and the post-merge live check is a tracked issue rather than a PR-body line. Refs: #7886 Co-Authored-By: Claude Opus 5.5 Claude-Session: https://claude.ai/code/session_019bQ8pL7fJgVy7YjQ7wK1Zw --- docs/conventions/pr-pipeline/CHANGELOG.md | 6 ++++++ docs/conventions/pr-pipeline/README.md | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/docs/conventions/pr-pipeline/CHANGELOG.md b/docs/conventions/pr-pipeline/CHANGELOG.md index c3124d40ef..91ea10a6fb 100644 --- a/docs/conventions/pr-pipeline/CHANGELOG.md +++ b/docs/conventions/pr-pipeline/CHANGELOG.md @@ -1,5 +1,11 @@ # Changelog for the PR pipeline convention +## 2.7.0 - 2026-10-18 + +[Changing this config](README.md#changing-this-config): a pull request that changes a lane and +lists a not-verified item that could stop the lane does not merge until the item is checked live, +and its post-merge live check is a tracked issue. `version` stays 2. + ## 2.6.1 - 2026-10-17 `address-feedback.mjs select` now also keeps a thread the lane escalated (the lanes App's last diff --git a/docs/conventions/pr-pipeline/README.md b/docs/conventions/pr-pipeline/README.md index 988862d098..92985430fa 100644 --- a/docs/conventions/pr-pipeline/README.md +++ b/docs/conventions/pr-pipeline/README.md @@ -873,6 +873,12 @@ The config file is a lane-power file: a human merges every change to it, and no same holds for the trusted-actor list and any change to a lane's permissions, tokens or triggers. Everything downstream of that human merge may propagate on its own. +A pull request that changes a lane and lists a not-verified item that could stop the lane from +running does not merge until that item is checked live. Its post-merge live check is a tracked +issue, not a line in the pull request body. Evidence: melodic-software/ci-workflows#755 listed its +intake author gate as "unverified, fails closed", merged, and skipped every intake run here +(melodic-software/ci-workflows#772). + ## Outside this repository The lanes depend on pieces other repositories own. Each row names where that piece is documented; From 8126d7a9fc44bf11db4cff34ff3c62400f7576eb Mon Sep 17 00:00:00 2001 From: Kyle Sexton <153232337+kyle-sexton@users.noreply.github.com> Date: Sun, 18 Oct 2026 00:31:12 -0400 Subject: [PATCH 2/2] docs(pr-pipeline): drop a leftover conflict marker from the changelog Co-Authored-By: Claude Opus 5.5 --- docs/conventions/pr-pipeline/CHANGELOG.md | 1 - 1 file changed, 1 deletion(-) diff --git a/docs/conventions/pr-pipeline/CHANGELOG.md b/docs/conventions/pr-pipeline/CHANGELOG.md index 8d0da83795..d8786a956d 100644 --- a/docs/conventions/pr-pipeline/CHANGELOG.md +++ b/docs/conventions/pr-pipeline/CHANGELOG.md @@ -50,7 +50,6 @@ an ancestor process's `/proc//environ`. `check-skill.sh` runs each checked `scripts/*.test.sh` and the checkout's markdownlint: checkout code with the network open, the power `run-test-file.sh` already gives the run ([pr-run-activity.md](pr-run-activity.md#skill-activities)). `version` stays 2. ->>>>>>> origin/main ## 2.6.1 - 2026-10-17