Skip to content

README: list the CATSTACK_DRAFTER_CORE flag - #743

Open
EdbertChan wants to merge 1 commit into
stack/EdbertChan/drafter-core-flag-readme/turn-drafter-core-pr-rules-off-unless-catstack--3014f5c0from
stack/EdbertChan/drafter-core-flag-readme/list-catstack-drafter-core-flag--11034a1e
Open

EdbertChan wants to merge 1 commit into
stack/EdbertChan/drafter-core-flag-readme/turn-drafter-core-pr-rules-off-unless-catstack--3014f5c0from
stack/EdbertChan/drafter-core-flag-readme/list-catstack-drafter-core-flag--11034a1e

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Summary

The readme's settings table now lists the new setting that turns the outside PR rule package back on.

Without this row, a reader of the table would not know the PR rules are off by default or how to turn them on.

Review Claim

The readme settings table lists the new PR rule setting and what it does.

Review Lane

docs

Review Unit

docs

Safety Invariant

Text-only change to one table row; no code, hook, or install path reads it.

Slice Rationale

Docs are their own review unit here, so this row sits on top of the PR that adds the setting.

Non-goals

  • Does not change any script or test.

Test Plan

Test Plan
  • python3 engine/skills/make-pr/scripts/preflight.py --base <parent branch> --body-file <this body>
  • Row reads: CATSTACK_DRAFTER_CORE=1 | env and files | the draft-pr scripts run the drafter-core PR rules; off, they print UNCHECKED and skip them

Revert Plan

Revert Plan
  • Safe to revert? Yes
  • Revert command: git revert <sha>
  • Post-revert steps: None
  • Data migration? No

🤖 Generated with Claude Code

Depends-On: #742

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

Copy link
Copy Markdown
Owner Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 Turn drafter-core PR rules off unless CATSTACK_DRAFTER_CORE=1 #742
2 README: list the CATSTACK_DRAFTER_CORE flag #743 👈

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