Skip to content

feat(interrogate): add docstring coverage checks to prek and lint - #20

Merged
dlstadther merged 1 commit into
mainfrom
dlstadther/interrogate
Sep 11, 2026
Merged

feat(interrogate): add docstring coverage checks to prek and lint#20
dlstadther merged 1 commit into
mainfrom
dlstadther/interrogate

Conversation

@dlstadther

Copy link
Copy Markdown
Owner

Summary

  • Add interrogate (docstring coverage) as a dev dependency, pinned to 1.7.0
  • Wire it into .pre-commit-config.yaml and make lint (used by nox's lint session)
  • Require 100% coverage (ignore-init-method = true, excluding tests/docs/noxfile.py)
  • Fill in missing module/class/method docstrings in sample_with_extra_install.py to pass the new check

Test plan

  • make lint passes (ruff, sqlfluff, interrogate)
  • make test passes
  • prek run interrogate --all-files passes

🤖 Generated with Claude Code

https://claude.ai/code/session_01KcSv8Fmjis7vAiVgkyHV14

Integrates interrogate at 100% required coverage, wired into the
pre-commit hooks and the make lint / nox lint targets. Fills in
missing docstrings in sample_with_extra_install.py to satisfy the
new check.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KcSv8Fmjis7vAiVgkyHV14
@dlstadther
dlstadther merged commit 1f3b9c1 into main Sep 11, 2026
5 checks passed
@dlstadther
dlstadther deleted the dlstadther/interrogate branch September 11, 2026 01:39
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