Merge pull request #59 from tjs-intel/master

MQuy · web-flow · commit 948212e0f157 · 2022-10-28T15:02:49.000+02:00
Fix failOnHint for unused exports
diff --git a/src/detect.js b/src/detect.js
@@ -11,7 +11,7 @@ function detectDeadCode(compilation, options) {
 	const includedFiles = fg.sync(getPattern(options));
 
 	let unusedFiles = [];
-	let unusedExportMap = [];
+	let unusedExportMap = {};
 
 	if (options.detectUnusedFiles) {
 		unusedFiles = includedFiles.filter(file => !compiledFiles[file]);
@@ -48,7 +48,7 @@ function detectDeadCode(compilation, options) {
 		}
 	}
 
-	if (unusedFiles.length > 0 || unusedExportMap.length > 0) {
+	if (unusedFiles.length > 0 || Object.keys(unusedExportMap).length > 0) {
 		if (options.failOnHint) {
 			process.exit(2);
 		}

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ function detectDeadCode(compilation, options) {`
`11`	`11`	`const includedFiles = fg.sync(getPattern(options));`
`12`	`12`
`13`	`13`	`let unusedFiles = [];`
`14`		`- let unusedExportMap = [];`
	`14`	`+ let unusedExportMap = {};`
`15`	`15`
`16`	`16`	`if (options.detectUnusedFiles) {`
`17`	`17`	`unusedFiles = includedFiles.filter(file => !compiledFiles[file]);`
`@@ -48,7 +48,7 @@ function detectDeadCode(compilation, options) {`
`48`	`48`	`}`
`49`	`49`	`}`
`50`	`50`
`51`		`- if (unusedFiles.length > 0 \|\| unusedExportMap.length > 0) {`
	`51`	`+ if (unusedFiles.length > 0 \|\| Object.keys(unusedExportMap).length > 0) {`
`52`	`52`	`if (options.failOnHint) {`
`53`	`53`	`process.exit(2);`
`54`	`54`	`}`