Improve a few names

webpro-nl · Oct 5, 2022 · 458cb5a · 458cb5a
1 parent 3a4c90d
commit 458cb5a
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 11 deletions.
diff --git a/README.md b/README.md
@@ -66,7 +66,8 @@ Options:
   --onlyExports                 Report only unused exports
   --onlyTypes                   Report only unused types
   --onlyDuplicates              Report only unused duplicate exports
-  --ignoreNamespaceImports      Ignore namespace imports (affects onlyFiles and onlyDuplicates)
+  --ignoreNamespaceImports      Ignore namespace imports (affects onlyExports and onlyTypes)
+  --noProgress                  Don't show dynamic progress updates
 
 Examples:
 

diff --git a/src/cli.ts b/src/cli.ts
@@ -17,7 +17,7 @@ const {
     onlyTypes,
     onlyDuplicates,
     ignoreNamespaceImports = false,
-    hideProgress = false
+    noProgress = false
   }
 } = parseArgs({
   allowPositionals: true,
@@ -29,7 +29,7 @@ const {
     onlyTypes: { type: 'boolean' },
     onlyDuplicates: { type: 'boolean' },
     ignoreNamespaceImports: { type: 'boolean' },
-    hideProgress: { type: 'boolean' }
+    noProgress: { type: 'boolean' }
   }
 });
 
@@ -42,7 +42,7 @@ const cwd = cwdArg ? path.resolve(cwdArg) : process.cwd();
 
 const configuration: Configuration = require(path.resolve(config));
 
-const isShowProgress = !hideProgress || !process.stdout.isTTY;
+const isShowProgress = !noProgress || !process.stdout.isTTY;
 const isFindAll = !onlyFiles && !onlyExports && !onlyTypes && !onlyDuplicates;
 const isFindUnusedFiles = onlyFiles === true || isFindAll;
 const isFindUnusedExports = onlyExports === true || isFindAll;
@@ -58,7 +58,7 @@ const main = async () => {
     isFindUnusedExports,
     isFindUnusedTypes,
     isFindDuplicateExports,
-    isIgnoreNamespaceImports: Boolean(ignoreNamespaceImports)
+    isFollowSymbols: !ignoreNamespaceImports
   });
 
   if (isFindUnusedFiles) logIssueGroupResult(cwd, 'UNUSED FILES', issues.file);

diff --git a/src/help.ts b/src/help.ts
@@ -8,8 +8,8 @@ Options:
   --onlyExports                 Report only unused exports
   --onlyTypes                   Report only unused types
   --onlyDuplicates              Report only unused duplicate exports
-  --ignoreNamespaceImports      Ignore namespace imports (affects onlyFiles and onlyDuplicates)
-  --hideProgress                Hide dynamic progress updates
+  --ignoreNamespaceImports      Ignore namespace imports (affects onlyExports and onlyTypes)
+  --noProgress                  Don't show dynamic progress updates
 
 Examples:
 

diff --git a/src/index.ts b/src/index.ts
@@ -17,7 +17,7 @@ export async function run(configuration: Configuration) {
     isFindUnusedExports,
     isFindUnusedTypes,
     isFindDuplicateExports,
-    isIgnoreNamespaceImports
+    isFollowSymbols
   } = configuration;
 
   // Create workspace for entry files + resolved dependencies
@@ -134,7 +134,7 @@ export async function run(configuration: Configuration) {
 
                 if (!isReferencedOnlyBySelf) return; // This identifier is used somewhere else
 
-                if (!isIgnoreNamespaceImports) {
+                if (isFollowSymbols) {
                   const symbol = identifier.getSymbol();
                   if (symbol) {
                     const referencingSourceFiles = sourceFile.getReferencingSourceFiles();

diff --git a/src/types.ts b/src/types.ts
@@ -9,7 +9,7 @@ export type Configuration = {
   isFindUnusedExports?: boolean;
   isFindUnusedTypes?: boolean;
   isFindDuplicateExports?: boolean;
-  isIgnoreNamespaceImports?: boolean;
+  isFollowSymbols?: boolean;
 };
 
 type FilePath = string;

diff --git a/test/index.spec.ts b/test/index.spec.ts
@@ -15,7 +15,7 @@ test('run', async () => {
     isFindUnusedExports: true,
     isFindUnusedTypes: true,
     isFindDuplicateExports: true,
-    isIgnoreNamespaceImports: false
+    isFollowSymbols: false
   });
 
   assert(issues.file.length === 1);