Skip to content

docs: align CONTRIBUTING.md coverage threshold with pyproject.toml (91%)#140

Merged
longieirl merged 1 commit intomainfrom
docs/align-contributing-coverage-threshold
Apr 10, 2026
Merged

docs: align CONTRIBUTING.md coverage threshold with pyproject.toml (91%)#140
longieirl merged 1 commit intomainfrom
docs/align-contributing-coverage-threshold

Conversation

@longieirl
Copy link
Copy Markdown
Owner

Pull Request

Summary

CONTRIBUTING.md claimed a 92% coverage requirement but the enforced threshold in packages/parser-core/pyproject.toml (fail_under = 91) and .github/PULL_REQUEST_TEMPLATE.md both say 91%. Aligns all three sources. Also corrects the integration snapshot description — the snapshot is committed to the repo (no personal data), not gitignored.

Changes

  • CONTRIBUTING.md: replace all 92%91% (5 occurrences: verify step, workflow step, coverage section, PR checklist, CI checks list)
  • CONTRIBUTING.md: fix integration snapshot description to reflect committed baseline

Type

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation
  • Performance
  • Security

Testing

  • Tests pass (coverage ≥ 91%)
  • Manually tested

Checklist

  • Code follows project style
  • Self-reviewed
  • Documentation updated (if needed)
  • No new warnings

Downstream impact

  • This PR changes a public interface in bankstatements_core (exported class, function, or exception)

…1% not 92%)

Fix integration snapshot description — snapshot is committed, not gitignored.
@longieirl longieirl self-assigned this Apr 10, 2026
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Apr 10, 2026
@longieirl longieirl merged commit 3345873 into main Apr 10, 2026
2 checks passed
@longieirl longieirl deleted the docs/align-contributing-coverage-threshold branch April 10, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants