Alan-Jowett / dagir / 20974123982

79%

push

github

Add coverage gate and policy documentation (#80)

* Initial plan

* feat: add coverage gate and document coverage policy

- Add coverage threshold check in workflow (max 2% decrease)
- Document coverage policy in CONTRIBUTING.md
- Include best practices for maintaining coverage
- Add instructions for viewing coverage locally

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* fix: add bc and curl to coverage workflow dependencies

- Ensure bc (basic calculator) is available for coverage math
- Ensure curl is available for Coveralls API calls

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* refactor: improve coverage gate robustness

- Add validation for current coverage extraction
- Add numeric validation for coverage values
- Add curl timeout and retry configuration
- Improve error messages and edge case handling
- Address code review feedback

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* fix: correct regex patterns for coverage extraction and validation

- Use consistent regex patterns for extraction and validation
- Support both whole numbers (85) and decimals (85.5)
- Prevent invalid formats like '85.' or '.5'
- Address final code review feedback

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* refactor: use jq for JSON parsing and improve pattern matching

- Add jq dependency for reliable JSON parsing
- Use jq to extract coverage from Coveralls API response
- Add URL encoding for branch name to prevent injection
- Improve lcov extraction with more specific pattern
- Address all code review feedback

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* fix: address PR review feedback

- Use POSIX grep -E instead of grep -P for portability
- Add bc -l flag for floating-point precision in coverage calculation
- URL-encode REPO_SLUG to prevent URL injection
- Add curl error logging with tee for debuggin... (continued)

2661 of 3673 branches covered (72.45%)

Branch coverage included in aggregate %.

1947 of 2165 relevant lines covered (89.93%)

4706396.32 hits per line