Skip to content

Remove Test Coverage section from README#38

Merged
Mossaka merged 2 commits intomainfrom
copilot/remove-test-coverage
Oct 30, 2025
Merged

Remove Test Coverage section from README#38
Mossaka merged 2 commits intomainfrom
copilot/remove-test-coverage

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 30, 2025

Removes duplicate test coverage information from README.md. Coverage details remain in TESTING.md.

Changes

  • Removed "Test Coverage" subsection (lines 98-107) containing coverage percentages and TESTING.md reference
  • README now flows directly from "Running Tests" to "Building" sections

The README previously displayed:

  • Logger: 100% coverage
  • Squid Config: 100% coverage
  • CLI Workflow: 100% coverage
  • Overall: 38%+ statement coverage

This information duplicates content already maintained in TESTING.md with full context and details.

Original prompt

remove Test Coverage
The project maintains comprehensive test coverage. Current coverage status:

Logger: 100% coverage
Squid Config: 100% coverage
CLI Workflow: 100% coverage
Overall: 38%+ statement coverage with established thresholds
For detailed testing information, see TESTING.md. from the README


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
@Mossaka Mossaka marked this pull request as ready for review October 30, 2025 19:20
@Mossaka Mossaka merged commit ebc2513 into main Oct 30, 2025
3 checks passed
@Mossaka Mossaka deleted the copilot/remove-test-coverage branch October 30, 2025 19:20
Copilot AI restored the copilot/remove-test-coverage branch October 30, 2025 19:20
Copilot AI changed the title [WIP] Remove test coverage section from documentation Remove Test Coverage section from README Oct 30, 2025
Copilot AI requested a review from Mossaka October 30, 2025 19:20
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.

2 participants