Skip to content

Mention CI in CONTRIBUTING's validation step - #70

Merged
dmccoystephenson merged 1 commit into
mainfrom
fix/contributing-mentions-ci
Jul 27, 2026
Merged

Mention CI in CONTRIBUTING's validation step#70
dmccoystephenson merged 1 commit into
mainfrom
fix/contributing-mentions-ci

Conversation

@dmccoystephenson

@dmccoystephenson dmccoystephenson commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

  • CONTRIBUTING.md's "Making a change" step 3 said "There is no automated test suite" and jumped straight to the manual behavioral checklist, without mentioning that .github/workflows/ci.yml runs scripts/check_docs.py automatically on every PR. A new contributor reading only CONTRIBUTING.md would have no idea their PR gets an automated doc-consistency check.
  • Now states what CI actually checks (placeholder/substitution-table sync, README Step-list 1:1, relative links) before noting it doesn't cover behavior.

Research grounding

Docs-only change, no template/phase behavior touched — no RESEARCH.md finding applies.

Test plan

  • python3 scripts/check_docs.py passes
  • No placeholders or Steps touched

drafted by Claude on behalf of Daniel Stephenson

New contributors had no indication that pushing a PR triggers an
automated doc-consistency check; CONTRIBUTING.md only described the
manual behavioral validation, which reads as if there's no automation
at all.
@dmccoystephenson
dmccoystephenson merged commit 0f4370c into main Jul 27, 2026
1 check passed
@dmccoystephenson
dmccoystephenson deleted the fix/contributing-mentions-ci branch July 27, 2026 01:55
dmccoystephenson added a commit that referenced this pull request Aug 3, 2026
New contributors had no indication that pushing a PR triggers an
automated doc-consistency check; CONTRIBUTING.md only described the
manual behavioral validation, which reads as if there's no automation
at all.
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