Skip to content

Commit b9febf8

Browse files
aristathclaude
andcommitted
Remove file limits from coverage details display
Show all files in the file-level coverage changes section instead of limiting to 10 files per category. This provides complete visibility into coverage changes across the entire codebase. Changes: - Removed .slice(0, 10) limits from new_files, improved, and degraded loops - Removed "... and X more" messages - All 146 files will now be shown in the details table 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 9aa6985 commit b9febf8

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

.github/workflows/code-coverage.yml

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -348,13 +348,10 @@ jobs:
348348
changesContent += '\n### 🆕 New Files\n\n';
349349
changesContent += '| Class | Coverage | Lines |\n';
350350
changesContent += '|-------|----------|-------|\n';
351-
for (const file of changesJson.new_files.slice(0, 10)) { // Limit to 10
351+
for (const file of changesJson.new_files) {
352352
const emoji = file.coverage >= 80 ? '🟢' : file.coverage >= 60 ? '🟡' : '🔴';
353353
changesContent += `| ${emoji} \`${file.class}\` | ${file.coverage.toFixed(2)}% | ${file.lines} |\n`;
354354
}
355-
if (changesJson.new_files.length > 10) {
356-
changesContent += `\n_... and ${changesJson.new_files.length - 10} more new files_\n`;
357-
}
358355
}
359356
360357
// Improved coverage
@@ -364,12 +361,9 @@ jobs:
364361
changesContent += '| Class | Before | After | Change |\n';
365362
changesContent += '|-------|--------|-------|--------|\n';
366363
const sortedImproved = changesJson.improved.sort((a, b) => b.diff - a.diff);
367-
for (const file of sortedImproved.slice(0, 10)) { // Limit to 10
364+
for (const file of sortedImproved) {
368365
changesContent += `| \`${file.class}\` | ${file.old.toFixed(2)}% | ${file.new.toFixed(2)}% | +${file.diff.toFixed(2)}% |\n`;
369366
}
370-
if (changesJson.improved.length > 10) {
371-
changesContent += `\n_... and ${changesJson.improved.length - 10} more improvements_\n`;
372-
}
373367
}
374368
375369
// Degraded coverage
@@ -379,12 +373,9 @@ jobs:
379373
changesContent += '| Class | Before | After | Change |\n';
380374
changesContent += '|-------|--------|-------|--------|\n';
381375
const sortedDegraded = changesJson.degraded.sort((a, b) => a.diff - b.diff);
382-
for (const file of sortedDegraded.slice(0, 10)) { // Limit to 10
376+
for (const file of sortedDegraded) {
383377
changesContent += `| \`${file.class}\` | ${file.old.toFixed(2)}% | ${file.new.toFixed(2)}% | ${file.diff.toFixed(2)}% |\n`;
384378
}
385-
if (changesJson.degraded.length > 10) {
386-
changesContent += `\n_... and ${changesJson.degraded.length - 10} more decreases_\n`;
387-
}
388379
}
389380
390381
// Wrap in collapsible details if there are changes

0 commit comments

Comments
 (0)