Skip to content

Include ci/ in coverage source for 100% requirement#3131

Merged
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/ci-coverage
Apr 26, 2026
Merged

Include ci/ in coverage source for 100% requirement#3131
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/ci-coverage

Conversation

@adamtheturtle
Copy link
Copy Markdown
Member

@adamtheturtle adamtheturtle commented Apr 25, 2026

Summary

  • Add ci/ to tool.coverage.run.source so ci/test_custom_linters.py is included in the 100% coverage requirement enforced by the coverage job.

Test plan

  • CI passes

Note

Low Risk
Low risk configuration change that expands coverage measurement to include ci/, which may cause CI to fail if those files are not fully exercised.

Overview
Coverage collection is expanded by adding ci/ to tool.coverage.run.source, making CI helper tests (e.g. ci/test_custom_linters.py) part of the 100% coverage gate.

test_tests_collected_once is slightly refactored to use dict.setdefault(...).add(...) when building the tests_to_patterns map (no functional change intended).

Reviewed by Cursor Bugbot for commit 3326f05. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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