Skip to content

Remove draft PR policy from agent guidance and Argos - #784

Merged
nedtwigg merged 2 commits into
mainfrom
excessive-drafts
Sep 24, 2026
Merged

nedtwigg merged 2 commits into
mainfrom
excessive-drafts

Conversation

@nedtwigg

@nedtwigg nedtwigg commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Remove the instruction for agents to open draft PRs, including the Workspaces testing guide.
  • Let Renovate use its normal PR state for routine and security updates.
  • Run Argos visual tests on draft PRs as well as ready PRs; keep the fork PR secret guard.
  • Keep the inactive Chromatic build integration intact.
  • Tighten the dor-cli.md wording to bring its preexisting word-budget overrun under the limit.

Validation

  • git diff --check passed.
  • Renovate JSON parses.
  • CI passed on 1124b3b, including Build & Test and both standalone platform checks.
  • Hosted PR preview passed.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Deploying mouseterm with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1124b3b
Status: ✅  Deploy successful!
Preview URL: https://9a6bdf0d.mouseterm.pages.dev
Branch Preview URL: https://excessive-drafts.mouseterm.pages.dev

View logs

@dormouse-bot dormouse-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The diff looks correct to me. Renovate's default is already draftPR: false, so dropping the explicit override on vulnerabilityAlerts changes nothing. Argos keeps its fork-secret guard. Nothing else in the repo still relies on the draft gating.

I'm not approving yet because Build & Test is red. It stops at docs/specs/dor-cli.md: 6314 words exceeds its 6300-word budget, and this PR doesn't touch that file. main fails the same way at 83694b8 (run), so the check should go green once main is fixed and merged in.

Copy link
Copy Markdown
Member Author

@dormouse-bot I fixed the preexisting dor-cli.md word-budget failure in 1124b3b by tightening the affected wording without changing the rules. The new Build & Test job has passed the spec lint gate and is still running the rest of the suite. I’ll check the final CI result here.

Copy link
Copy Markdown
Member Author

@dormouse-bot Follow-up: CI run 36033054749 completed successfully on 1124b3b. Build & Test and all platform checks are green. Thanks for the review.

@nedtwigg
nedtwigg merged commit 8596e30 into main Sep 24, 2026
12 checks passed
@nedtwigg
nedtwigg deleted the excessive-drafts branch September 24, 2026 17:22

This branch is waiting to be deployed

1 waiting deployment
hosted-preview — 1124b3bc Waiting Sep 24, 2026 by nedtwigg via cleanup #373
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.

2 participants