Skip to content

ci: add coverage threshold gate (40% minimum)#754

Merged
ArangoGutierrez merged 2 commits intoNVIDIA:mainfrom
ArangoGutierrez:release/v0.3.0-coverage-threshold
Mar 30, 2026
Merged

ci: add coverage threshold gate (40% minimum)#754
ArangoGutierrez merged 2 commits intoNVIDIA:mainfrom
ArangoGutierrez:release/v0.3.0-coverage-threshold

Conversation

@ArangoGutierrez
Copy link
Copy Markdown
Collaborator

Summary

  • Adds coverage threshold check (40% minimum) after make coverage in CI
  • Uses go tool cover -func to extract total coverage, fails with ::error:: annotation if below threshold
  • Conservative initial threshold since PR build: expand test coverage scope to cmd/, api/, and internal/ #750 just expanded coverage scope to include cmd/, api/, internal/
  • Threshold can be raised incrementally as coverage improves

Test plan

  • Verify YAML syntax is correct
  • Confirm step runs after coverage generation and before Coveralls upload
  • Verify bc is available on ubuntu-latest runners

Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
@ArangoGutierrez ArangoGutierrez marked this pull request as ready for review March 30, 2026 12:24
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23744430291

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 45.802%

Totals Coverage Status
Change from base Build 23740406589: 0.0%
Covered Lines: 4975
Relevant Lines: 10862

💛 - Coveralls

Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
@ArangoGutierrez ArangoGutierrez merged commit 2f61797 into NVIDIA:main Mar 30, 2026
17 checks passed
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