Skip to content

Commit 8e0fbe3

Browse files
Add explicit PR template instructions in issue body and agent protocol
Address review feedback: - Add '### PR template' section in generated issue body directing the agent to use ?template=coverage.md or copy template contents - Update copilot-test-coverage.instructions.md PR requirements with explicit template usage instructions - Validated script locally with mock data: below-threshold, above- threshold, and absolute-path normalization all pass correctly Agent-Logs-Url: https://github.com/microsoft/vscode-cmake-tools/sessions/cb96c55e-0fc9-4e59-b0a8-a1fc704e2182 Co-authored-by: hanniavalera <90047725+hanniavalera@users.noreply.github.com>
1 parent a402a11 commit 8e0fbe3

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

.github/instructions/copilot-test-coverage.instructions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,5 +73,5 @@ Do **not** open the PR unless every file listed in the issue has either:
7373

7474
- Branch name: `coverage/<module-name>-tests`
7575
- Open as **ready for review** only after the self-audit checklist in the issue is fully checked
76-
- PR description must use `.github/PULL_REQUEST_TEMPLATE/coverage.md`
76+
- **PR description must use the coverage template**: copy the contents of `.github/PULL_REQUEST_TEMPLATE/coverage.md` into the PR body (or append `?template=coverage.md` to the PR creation URL). Fill in the coverage-delta table and check every self-audit box.
7777
- `CHANGELOG.md` must have one entry under `Improvements:`

.github/scripts/check-coverage-and-open-issue.mjs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,13 @@ New tests **must** go in \`test/unit-tests/backend/\` and run in plain Node.js (
150150
- Pure logic modules (\`expand.ts\`, \`shlex.ts\`, \`cache.ts\`, \`diagnostics/\`, \`preset.ts\`) are ideal targets.
151151
${remainingNote}
152152
153+
### PR template
154+
155+
When opening the pull request, **you must use the coverage PR template**.
156+
Include \`?template=coverage.md\` in the PR creation URL, or copy the contents of
157+
\`.github/PULL_REQUEST_TEMPLATE/coverage.md\` into the PR description and fill in
158+
the coverage table and checklist.
159+
153160
### Constraints
154161
155162
- Tests go in \`test/unit-tests/backend/\` — use Mocha \`suite\`/\`test\` with \`assert\`

0 commit comments

Comments
 (0)