Skip to content

docs: propose auto-approve conditions for docs PRs - #1186

Draft
teallarson wants to merge 1 commit into
mainfrom
teallarson/auto-approve
Draft

docs: propose auto-approve conditions for docs PRs#1186
teallarson wants to merge 1 commit into
mainfrom
teallarson/auto-approve

Conversation

@teallarson

@teallarson teallarson commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Records the risk tiers we want to sort docs PRs into, ahead of building the workflow that enforces them.

Nothing is enforced by this PR — it's a proposal to react to. The tiers:

  • Low risk (auto-approvable) — human-authored PRs under 300 lines with no new routes
  • Medium risk (any team member) — bot-authored PRs; human PRs adding a page or over 300 lines
  • Higher risk (codeowner) — the toolkit gen pipeline, GitHub Actions, package.json

Merging stays a human action either way, as in the monorepo and www.

Placed at .github/auto-approve/README.md rather than CONTRIBUTING.md so the policy sits next to the mechanism once the mechanism exists — the same layout the monorepo uses for .github/autoapprove/. Happy to move it if you'd rather it be more visible to contributors.

Fuller writeup with risks, tradeoffs, and the alternatives we considered: Google Doc

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only policy proposal with no workflow or runtime behavior changes.

Overview
Adds .github/auto-approve/README.md to document a proposed risk tiering policy for docs PRs, ahead of any automation.

Low risk (future auto-approve): human-authored changes under 300 lines with no new routes. Medium: bot PRs or human PRs that add a page or exceed 300 lines (any teammate review). Higher: toolkit gen pipeline, GitHub Actions, or package.json (codeowner review). The doc states merging stays manual and aligns with practices in other Arcade repos; this PR does not add workflows or enforce anything.

Reviewed by Cursor Bugbot for commit c23fc6c. Bugbot is set up for automated code reviews on this repo. Configure here.

Records the risk tiers we want to sort docs PRs into, ahead of building
the workflow that enforces them. Sits under .github/auto-approve/ so the
policy lands next to the mechanism once that mechanism exists.

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

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 9, 2026 5:23pm UTC

Request Review

@teallarson
teallarson marked this pull request as ready for review September 9, 2026 17:22
@teallarson
teallarson marked this pull request as draft September 9, 2026 17:24
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