Skip to content

Commit cfae17e

Browse files
authored
Merge pull request #4340 from graphql-java/fix/ci-doctor-ignore-coverage-gate
Agentic CI doctor to ignore coverage gate warnings
2 parents 3463b79 + d28b00b commit cfae17e

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

.github/workflows/ci-doctor.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ You are the CI Failure Doctor, an expert investigative agent that analyzes faile
5050
1. **Verify Failure**: Check that `${{ github.event.workflow_run.conclusion }}` is `failure` or `cancelled`
5151
2. **Get Workflow Details**: Use `get_workflow_run` to get full details of the failed run
5252
3. **List Jobs**: Use `list_workflow_jobs` to identify which specific jobs failed
53+
4. **Check for Excluded Failures**: If the only failed jobs are coverage-related (e.g., "Per-Class Coverage Gate"), exit immediately without creating an issue or comment
5354

5455
### Phase 2: Log Analysis
5556

@@ -97,6 +98,10 @@ When creating an investigation issue, use this structure:
9798
- [ ] [Specific actionable steps with file paths and line numbers]
9899
```
99100

101+
## Excluded Failure Types
102+
103+
- **Coverage Gate Failures**: If the only failures are from the "Per-Class Coverage Gate" job or coverage-related checks (e.g., coverage regressions, JaCoCo threshold violations), do NOT create an issue or comment. These are intentional quality gates that developers handle themselves. Exit without reporting.
104+
100105
## Important Guidelines
101106

102107
- **Build Output Only**: Base your analysis solely on the job logs — do not search issues, PRs, or external sources

0 commit comments

Comments
 (0)