Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit db1183e

Browse files
chore: refine comment format for test coverage reviews in workflow
1 parent 8004b9a commit db1183e

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

.github/workflows/test-check.yaml

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -142,22 +142,26 @@ jobs:
142142
NEEDS UPDATES: Changes that make existing test assertions incorrect or incomplete
143143
NO CHANGES: Tests already sufficient, or changes don't affect testable behavior
144144
145-
6. Comment on the PR using EXACTLY this format (no additional sections)
145+
6. Comment on the PR using this format (only include the section relevant to the findings)
146146
147147
COMMENT FORMAT:
148148
## 🧪 Test Coverage Review
149149
150+
If tests are needed:
150151
### ❌ Tests Needed
151-
- **path/to/file.go** - New function needs unit tests
152+
- **path/to/file.go** - Brief description of what needs testing
152153
154+
If tests need updates:
153155
### ⚠️ Tests Need Updates
154-
- **path/to/file_test.go** - Assertions outdated for changed behavior
156+
- **path/to/file_test.go** - Brief description of what needs updating
155157
158+
If tests should be removed:
156159
### ⚠️ Tests to Remove
157-
- **path/to/old_test.go** - Tests orphaned deleted code
160+
- **path/to/old_test.go** - Brief description of why
158161
162+
If no changes needed:
159163
### ✅ No Changes Needed
160-
[Reason: Tests already updated in PR | Internal refactor with existing coverage | Docs-only changes | Config-only changes]
164+
Brief reason (e.g., Tests already updated in PR | Internal refactor with existing coverage | Docs-only changes | Config-only changes)
161165
162166
---
163167
*This comment was generated by an AI Agent through [Coder Tasks](https://coder.com/docs/ai-coder/tasks)*

0 commit comments

Comments
 (0)