refactor(format): combines two maps into one

which is good for readability (and arguably a bit for performance, even though at the scale of the inputs it won't be perceptible to really matter)
sverweij · Oct 3, 2024 · 0f61590 · 0f61590
1 parent e0b5b5c
commit 0f61590
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/dist/format/format.js b/dist/format/format.js
diff --git a/src/cli.ts b/src/cli.ts
@@ -33,7 +33,7 @@ export async function cli(
   pOutStream: Writable = process.stdout,
   pErrorStream: Writable = process.stderr,
   pErrorExitCode: number = 1,
-) {
+): Promise<void> {
   try {
     const lArguments = getArguments(pArguments);
 

diff --git a/src/format/format.ts b/src/format/format.ts
@@ -16,10 +16,7 @@ export function format(
   pExtensions: string,
 ): string {
   const lExtensions: Set<string> = new Set(
-    pExtensions
-      .split(",")
-      .map((pExtension) => pExtension.trim())
-      .map((pExtension) => `.${pExtension}`),
+    pExtensions.split(",").map((pExtension) => `.${pExtension.trim()}`),
   );
   // @ts-expect-error ts(2722) - Object is possibly 'undefined' - that's not
   // possible // because the OUTPUT_TYPE_TO_FUNCTION map contains all possible