From 16689f4d503040c7d5f15e3da2b2e9d31b679e0d Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Thu, 21 Nov 2024 11:43:23 +0900 Subject: [PATCH] Fix api-reports space-after-comma bug Signed-off-by: Juan Escalada --- .../repo-tools/src/commands/api-reports/api-reports.ts | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/packages/repo-tools/src/commands/api-reports/api-reports.ts b/packages/repo-tools/src/commands/api-reports/api-reports.ts index 637cd86671..0603188dac 100644 --- a/packages/repo-tools/src/commands/api-reports/api-reports.ts +++ b/packages/repo-tools/src/commands/api-reports/api-reports.ts @@ -45,7 +45,15 @@ export const buildApiReports = async (paths: string[] = [], opts: Options) => { const runTsc = opts.tsc; const allowWarnings = parseArrayOption(opts.allowWarnings); const allowAllWarnings = opts.allowAllWarnings; - const omitMessages = parseArrayOption(opts.omitMessages); + const omitMessagesRaw = opts.omitMessages; + + if (omitMessagesRaw && omitMessagesRaw.endsWith(',')) { + throw new Error( + `Invalid value for --omit-messages: ${omitMessagesRaw}\nMust be a comma-separated list of strings without spaces or wrapped in quotations.`, + ); + } + + const omitMessages = parseArrayOption(omitMessagesRaw); const isAllPackages = !paths?.length; const selectedPackageDirs = await resolvePackagePaths({