Fix errors and update formatting

RSeidelsohn · Aug 30, 2024 · c78518f · c78518f
1 parent da24ced
commit c78518f
Show file tree

Hide file tree

Showing 4 changed files with 183 additions and 174 deletions.
diff --git a/README.md b/README.md
@@ -11,6 +11,7 @@
     -   [Table of Contents](#table-of-contents)
     -   [Introduction](#introduction)
     -   [Changes](#changes)
+        -   [Version 4.4.1](#Version-4-4-1)
         -   [Version 4.4.0](#Version-4-4-0)
         -   [Version 4.3.1](#Version-4-3-1)
         -   [Version 4.3.0](#Version-4-3-0)
@@ -127,6 +128,10 @@ You could see something like this:
 
 ## <a id="changes"></a>Changes (see a more detailed and always up-to-date list [here](https://github.com/RSeidelsohn/license-checker-rseidelsohn/releases))
 
+### <a id="Version-4-4-1"></a>Version 4.4.1
+
+fix: Fix errors that broke the whole new version (sorry for these)
+
 ### <a id="Version-4-4-0"></a>Version 4.4.0
 
 chore(deps-dev): bump braces from 3.0.2 to 3.0.3 by @dependabot in https://github.com/RSeidelsohn/license-checker-rseidelsohn/pull/114

diff --git a/lib/args.js b/lib/args.js
@@ -10,7 +10,7 @@ import path from 'node:path';
 
 const knownOptions = {
     angularCli: Boolean,
-    clarificationsFile: path,clarificationsMatchAll
+    clarificationsFile: path,
     clarificationsMatchAll: Boolean,
     color: Boolean,
     csv: Boolean,

diff --git a/lib/index.js b/lib/index.js
@@ -46,8 +46,10 @@ const recursivelyCollectAllDependencies = (options) => {
     let licenseData;
     let licenseFile;
     let noticeFiles = [];
-    const clarification = options.clarifications[currentExtendedPackageJson.name]?.find((clarification) => 
-        currentExtendedPackageJson.version == clarification.semverRange || semver.satisfies(currentExtendedPackageJson.version, clarification.semverRange)
+    const clarification = options.clarifications[currentExtendedPackageJson.name]?.find(
+        (clarification) =>
+            currentExtendedPackageJson.version == clarification.semverRange ||
+            semver.satisfies(currentExtendedPackageJson.version, clarification.semverRange),
     );
     let passedClarificationCheck = clarification?.checksum ? false : true;
     if (clarification) {
@@ -411,7 +413,7 @@ const init = (args, callback) => {
                 const semverRange = versionString.slice(versionSplit + 1);
                 clarifications[name] ??= [];
                 // keep track for each clarification if it was used, optionally error when not
-                clarifications[name].push({...clarification, semverRange, used: false});
+                clarifications[name].push({ ...clarification, semverRange, used: false });
             }
         }
     }
@@ -449,20 +451,22 @@ const init = (args, callback) => {
             unknown: args.unknown,
             currentRecursionDepth: 0,
             clarifications,
-        })
+        });
 
         if (args.clarificationsMatchAll) {
-            const unusedClarifications = []
-            for (const [package, entries] of Object.entries(clarifications)) {
+            const unusedClarifications = [];
+            for (const [packageName, entries] of Object.entries(clarifications)) {
                 for (const clarification of entries) {
                     if (!clarification.used) {
-                        unusedClarifications.push(`${package}@${clarification.semverRange}`);
+                        unusedClarifications.push(`${packageName}@${clarification.semverRange}`);
                     }
                 }
             }
             if (unusedClarifications.length) {
                 console.error(
-                    `Some clarifications (${unusedClarifications.join(', ')}) were unused and --clarificationsMatchAll was specified. Exiting.`,
+                    `Some clarifications (${unusedClarifications.join(
+                        ', ',
+                    )}) were unused and --clarificationsMatchAll was specified. Exiting.`,
                 );
 
                 process.exit(1);