From 4cec99ed1360b0908ddefa01cd211fe97adb836a Mon Sep 17 00:00:00 2001 From: Derek Cofausper <256792747+decofe@users.noreply.github.com> Date: Wed, 25 Feb 2026 13:58:39 -0800 Subject: [PATCH] chore(bench): include core count in Slack notification when non-default (#22584) Co-authored-by: Amp --- .github/scripts/bench-slack-notify.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/scripts/bench-slack-notify.js b/.github/scripts/bench-slack-notify.js index 66fc23e0e6..0a6334b4e4 100644 --- a/.github/scripts/bench-slack-notify.js +++ b/.github/scripts/bench-slack-notify.js @@ -118,9 +118,12 @@ function buildSuccessBlocks({ summary, prNumber, actor, actorSlackId, jobUrl, re if (fl2) featureLine += ` | <${fl2}|Samply 2>`; const warmup = summary.warmup_blocks || process.env.BENCH_WARMUP_BLOCKS || ''; - const countsLine = warmup - ? `*Warmup:* ${warmup} | *Blocks:* ${summary.blocks}` - : `*Blocks:* ${summary.blocks}`; + const cores = process.env.BENCH_CORES || '0'; + const countsParts = []; + if (warmup) countsParts.push(`*Warmup:* ${warmup}`); + countsParts.push(`*Blocks:* ${summary.blocks}`); + if (cores !== '0') countsParts.push(`*Cores:* ${cores}`); + const countsLine = countsParts.join(' | '); const sectionText = [metaParts.join(' | '), '', baselineLine, featureLine, countsLine].join('\n');