Fix build script --report option (#15323)

sosukesuzuki · fisker · web-flow · commit a73cfa0ec43d · 2023-08-29T14:50:01.000+08:00
Co-authored-by: fisker Cheung &lt;lionkay@gmail.com&gt;
diff --git a/scripts/build/README.md b/scripts/build/README.md
@@ -79,8 +79,6 @@ yarn build --report=all
 yarn build --report=stdout --report=text --report=html
 ```
 
-**`--report` equals to `--report=html`**
-
 ### `--minify` and `--no-minify`
 
 By default, the file minification is controlled by `config.mjs` and `bundler.mjs`, these flags are added to override that behavior.
diff --git a/scripts/build/parse-arguments.js b/scripts/build/parse-arguments.js
@@ -30,7 +30,7 @@ function parseArguments() {
     minify: values.minify ? true : values["no-minify"] ? false : undefined,
     clean: values.clean,
     saveAs: values["save-as"],
-    report: values.report,
+    reports: values.report,
   };
 
   if (result.saveAs) {
@@ -59,14 +59,14 @@ function parseArguments() {
     }
   }
 
-  if (Array.isArray(result.report) && result.report.includes("all")) {
-    if (result.report.length !== 1) {
+  if (Array.isArray(result.reports) && result.reports.includes("all")) {
+    if (result.reports.length !== 1) {
       throw new Error(
         "'--report=all' can not use with another '--report' flag",
       );
     }
 
-    result.report = ["html", "text", "stdin"];
+    result.reports = ["html", "text", "stdout"];
   }
 
   return result;

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ function parseArguments() {`
`30`	`30`	`minify: values.minify ? true : values["no-minify"] ? false : undefined,`
`31`	`31`	`clean: values.clean,`
`32`	`32`	`saveAs: values["save-as"],`
`33`		`- report: values.report,`
	`33`	`+ reports: values.report,`
`34`	`34`	`};`
`35`	`35`
`36`	`36`	`if (result.saveAs) {`
`@@ -59,14 +59,14 @@ function parseArguments() {`
`59`	`59`	`}`
`60`	`60`	`}`
`61`	`61`
`62`		`- if (Array.isArray(result.report) && result.report.includes("all")) {`
`63`		`- if (result.report.length !== 1) {`
	`62`	`+ if (Array.isArray(result.reports) && result.reports.includes("all")) {`
	`63`	`+ if (result.reports.length !== 1) {`
`64`	`64`	`throw new Error(`
`65`	`65`	`"'--report=all' can not use with another '--report' flag",`
`66`	`66`	`);`
`67`	`67`	`}`
`68`	`68`
`69`		`- result.report = ["html", "text", "stdin"];`
	`69`	`+ result.reports = ["html", "text", "stdout"];`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`return result;`