Skip to content

Commit e2e1781

Browse files
authored
Merge pull request #33622 from storybookjs/norbert/fix-discord-report
Build: Fix failure discord message CI workflow helper
2 parents af8efd6 + b6bf0df commit e2e1781

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

scripts/bench/bench-packages.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -410,7 +410,10 @@ const run = async () => {
410410
.option(
411411
'-p, --pull-request <number>',
412412
'The PR number to add compare results to. Only used together with --baseBranch',
413-
function parseInt(value) {
413+
function parsePrNumber(value) {
414+
if (value === '0' || value === '') {
415+
return null;
416+
}
414417
const parsedValue = Number.parseInt(value);
415418
if (Number.isNaN(parsedValue)) {
416419
throw new InvalidArgumentError('Must be a number');

scripts/ci/utils/helpers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ export const workflow = {
224224
'discord/status': {
225225
only_for_branches: ['main', 'next', 'next-release', 'latest-release'].join(','),
226226
fail_only: true,
227-
failure_message: `yarn get-report-message ${workflow} ${template}`,
227+
failure_message: `$(yarn get-report-message ${workflow} ${template})`,
228228
},
229229
},
230230
];

0 commit comments

Comments
 (0)