Improvement: Raise CI coverage threshold to 90% + enable branch coverage
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
Research direction
Open .github/workflows/code_coverage.yml and inspect the current coverage workflow parameters. Confirm Tasks 44, 45, and 46 have brought all components to 90% line and branch coverage before making the change. Done means CI enforces a 90% minimum and generates branch coverage reports without breaking pull requests.
Written by the indexing model from the issue text.
Description
What
After Tasks
- https://github.com/eclipse-score/inc_time/issues/44
- https://github.com/eclipse-score/inc_time/issues/45
- https://github.com/eclipse-score/inc_time/issues/46
are complete and all components reach 90% line +
90% branch coverage, enforce the new threshold in the CI pipeline.
Update .github/workflows/code_coverage.yml:
# Before:
min-coverage: 85
# After:
min-coverage: 90
genhtml-extra-flags: "--branch-coverage"
Why
Raising the threshold without first reaching it (Tasks 5a–5c) would break
all PRs. This task is the final enforcement step — it gates the CI against
regressions below 90%.
References
- Current coverage workflow:
https://github.com/eclipse-score/inc_time/blob/main/.github/workflows/code_coverage.yml - Reusable workflow parameters (
min-coverage,genhtml-extra-flags):
https://github.com/eclipse-score/cicd-workflows/blob/main/.github/workflows/cpp-coverage.yml
Estimates for realization
S - Day
Category
- Affects Detailed Design
Requirements / Architecture
- Requirements / Architecture are not affected by this change?
- Dominant language
- C++
- Stars
- 2
- Forks
- 12
- Avg merge
- 4d 4h
- Merged PRs (30d)
- 21
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from eclipse-score/time
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 65/100
eclipse-score/time#226 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
eclipse-score/time#213 ·
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
eclipse-score/time#212 ·
-
Release v0.10 Open
Difficulty 5/5 Over a week Newbie friendliness 20/100
eclipse-score/time#211 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 55/100
eclipse-score/time#180 ·
All issues in eclipse-score/time
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
NVIDIA/cuda-samples#453 ·