Skip to content

docs: enforce public docstring coverage via ruff D1 and backfill#26

Merged
jeqcho merged 1 commit into
mainfrom
docs/public-docstring-gate
Jul 14, 2026
Merged

docs: enforce public docstring coverage via ruff D1 and backfill#26
jeqcho merged 1 commit into
mainfrom
docs/public-docstring-gate

Conversation

@jeqcho

@jeqcho jeqcho commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Adds Ruff's pydocstyle presence rules (D1) to the lint gate (D105/D107 exempt; tests and the mkdocs-gen-files build script under scripts/ excluded) and backfills the two missing public docstrings in cli.py. Public docstring coverage (interrogate, excluding private/nested/init): 80.0% -> 100.0%.

Documented in CLAUDE.md, README, and docs/contributing.md (so external benchmark PRs see the requirement), with the gate-backed docs-coverage badge in the README.

All gates green locally: ruff check, ruff format, strict mypy, pytest at 100% coverage, mkdocs build --strict.

Part of the org-wide rollout. Implemented by Codex (GPT), reviewed by Fable.

🤖 Generated with Claude Code

Enable Ruff D1 for public API symbols and backfill CLI contracts so missing public docstrings fail the quality gate. Exempt tests and build scripts from the public API check, document contributor expectations, and advertise the 100% gate in the README badge block.

Interrogate coverage: 80.0% before, 100.0% after.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jeqcho
jeqcho merged commit 5a38076 into main Jul 14, 2026
12 checks passed
@jeqcho
jeqcho deleted the docs/public-docstring-gate branch July 14, 2026 01:19
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