Skip to content

Commit b4e086c

Browse files
andimarekclaude
andcommitted
Make per-class coverage table more compact
Strip $InnerClass suffixes from class names and remove emoji from per-class delta cells. The +/- sign already conveys direction. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent cc22c5e commit b4e086c

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

.github/workflows/pull_request.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -302,16 +302,15 @@ jobs:
302302
changedClasses.sort((a, b) => a.name.localeCompare(b.name));
303303
304304
function shortenClass(name) {
305-
const dollarIdx = name.indexOf('$');
306-
const mainPart = dollarIdx >= 0 ? name.substring(0, dollarIdx) : name;
307-
const suffix = dollarIdx >= 0 ? name.substring(dollarIdx) : '';
305+
const mainPart = name.replace(/\$.*$/, '');
308306
const parts = mainPart.split('.');
309307
const shortened = parts.map((p, i) => i < parts.length - 1 ? p[0] : p);
310-
return shortened.join('.') + suffix;
308+
return shortened.join('.');
311309
}
312310
313311
function fmtClassDelta(delta) {
314-
return fmtPctDelta(delta, 0);
312+
if (Math.abs(delta) < 0.05) return '±0.0%';
313+
return delta > 0 ? `+${delta.toFixed(1)}%` : `${delta.toFixed(1)}%`;
315314
}
316315
317316
if (changedClasses.length > 0) {

0 commit comments

Comments
 (0)