Skip to content

Commit a233572

Browse files
l0lawrenceCopilot
andcommitted
Drop per-file list from emitter-diff PR comment; link the HTML instead
The sticky comment now shows only the one-line summary (file count, +/-) and the emitter-diff-html artifact link, not an enumerated list of changed files. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 174b1cb commit a233572

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

.github/workflows/ci-emitter-diff-python.yml

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ name: "python / emitter diff"
33
# Generates code with the current checkout's emitter and with the approved
44
# baseline emitter (the commit SHA in eng/emitter-diff/baselines/python.sha),
55
# then diffs the two. The rendered HTML diff is uploaded as a per-PR
6-
# artifact, and a sticky PR comment summarizes the changed files (+/-) with a
7-
# link to download it, so reviewers can see exactly how an emitter change
6+
# artifact, and a sticky PR comment summarizes the change (file count, +/-)
7+
# with a link to download it, so reviewers can see exactly how an emitter change
88
# affects generated SDKs. Powered by the language-agnostic eng/emitter-diff tool.
99
#
1010
# Python's generator uses a native two-phase pipeline (TypeSpec emits YAML, then
@@ -176,11 +176,6 @@ jobs:
176176
body += `✅ **No changes** — generated output matches the approved baseline.\n`;
177177
} else {
178178
body += `❌ **${files.length} file(s) changed** · +${insertions} / -${deletions}\n\n`;
179-
const shown = files.slice(0, 50);
180-
body += `<details><summary>Changed files</summary>\n\n` +
181-
shown.map((f) => `- \`${f}\``).join("\n") +
182-
(files.length > shown.length ? `\n- …and ${files.length - shown.length} more` : "") +
183-
`\n</details>\n\n`;
184179
if (htmlExists) {
185180
body += `📄 Download the **emitter-diff-html** artifact from the ` +
186181
`[workflow run](${runUrl}) for the full side-by-side rendered diff.\n\n`;

0 commit comments

Comments
 (0)