From 22e482b79518a55dc6d5f84ab5235945b8fa8f47 Mon Sep 17 00:00:00 2001 From: Alan Clucas Date: Mon, 10 Aug 2026 12:43:59 +0000 Subject: [PATCH] ci: enable the PR readiness helper The helper was merged with DRY_RUN on so its decisions could be verified from job summaries on real PRs. Flip DRY_RUN off so it now posts its sticky comment and drafts not-ready PRs for real. Signed-off-by: Alan Clucas Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01GYsLgxBxPGMmJYyfmXdvaJ --- .github/pr-readiness/README.md | 4 ++-- .github/workflows/pr-readiness.yaml | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/pr-readiness/README.md b/.github/pr-readiness/README.md index 695d572ab075..b844806b5224 100644 --- a/.github/pr-readiness/README.md +++ b/.github/pr-readiness/README.md @@ -46,9 +46,9 @@ A deterministic check ([`template.ts`](template.ts)) compares the description ag - PR title/body/branch are attacker-controlled: they are only ever handled as data, never interpolated into shell or scripts. The comment never echoes contributor-supplied text — only the bot's own guidance, check titles/URLs, and template section names. - All actions are pinned to full commit SHAs (enforced by repo lint). -## Dry run & rollout +## Dry run -`DRY_RUN: "true"` in the workflow renders the would-be comment and decisions to the job's **step summary** instead of commenting or drafting. Roll out: merge with dry-run on → watch summaries on real PRs (correct PR resolution, author gating, sensible text) → flip to `"false"`. +Setting `DRY_RUN: "true"` in the workflow renders the would-be comment and decisions to the job's **step summary** instead of commenting or drafting. Use it to test changes to the bot against real PRs (correct PR resolution, author gating, sensible text) without posting. ## Maintenance notes diff --git a/.github/workflows/pr-readiness.yaml b/.github/workflows/pr-readiness.yaml index dad71a71c460..bf8da7d58779 100644 --- a/.github/workflows/pr-readiness.yaml +++ b/.github/workflows/pr-readiness.yaml @@ -26,9 +26,9 @@ concurrency: cancel-in-progress: false env: - # Dry-run: render to the job summary instead of commenting/drafting. - # Flip to "false" to go live. - DRY_RUN: "true" + # Dry-run: set to "true" to render to the job summary instead of + # commenting/drafting. + DRY_RUN: "false" jobs: assess: