@knime/utils 1.3.2 → 1.3.3

package/CHANGELOG.md

@@ -1,10 +1,17 @@
 # @knime/utils
 
+## 1.3.3
+
+### Patch Changes
+
+- Updated dependencies [4bda98c]
+  - @knime/styles@1.4.0
+
 ## 1.3.2
 
 ### Patch Changes
 
-- ee229d3: add isWindows check to navigator utils
+- bb186d1: add isWindows check to navigator utils
 
 ## 1.3.1
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knime/utils",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "Internal utility functions and constants shared across @knime packages.",
   "homepage": "https://knime.github.io/webapps-common/",
   "license": "GPL 3 and Additional Permissions according to Sec. 7 (SEE the file LICENSE)",
@@ -9,7 +9,7 @@
   "dependencies": {
     "date-fns": "2.30.0",
     "date-fns-tz": "2.0.0",
-    "@knime/styles": "1.3.1"
+    "@knime/styles": "1.4.0"
   },
   "peerDependencies": {
     "consola": "3.x"

package/src/config/globals.d.ts

@@ -1,3 +1,9 @@
-/* eslint-disable one-var */
+import type { ConsolaInstance } from "consola";
 
-declare const consola: import("consola").ConsolaInstance;
+declare global {
+  const consola: ConsolaInstance;
+
+  interface Window {
+    consola: ConsolaInstance;
+  }
+}

package/src/filters.ts

@@ -65,7 +65,7 @@ const modeDefinitions = [
       try {
         const flags = caseSensitiveSearch ? "" : "i";
         return new RegExp(searchTerm, flags);
-      } catch (error) {
+      } catch (_error) {
         // In case of an invalid regular expression, an impossible
         // regex is returned, not matching anything.
         return /$^/;
@@ -82,7 +82,7 @@ const modeDefinitions = [
       try {
         const flags = caseSensitiveSearch ? "" : "i";
         return new RegExp(`^(${searchTerm})$`, flags);
-      } catch (error) {
+      } catch (_error) {
         // In case of an invalid regular expression, an impossible
         // regex is returned, not matching anything.
         return /$^/;

package/src/format.ts

@@ -13,7 +13,7 @@ export const formatDateString = (
 ) => {
   const date = new Date(dateString);
 
-  // @ts-ignore
+  // @ts-expect-error Argument of type 'Date' is not assignable to parameter of type 'number'
   if (isNaN(date)) {
     throw Error("Invalid Date format");
   }
@@ -47,7 +47,7 @@ export const formatTimeString = (
 ) => {
   const time = new Date(timeString);
 
-  // @ts-ignore
+  // @ts-expect-error Argument of type 'Date' is not assignable to parameter of type 'number'
   if (isNaN(time)) {
     throw Error("Invalid Date format");
   }
@@ -79,7 +79,6 @@ export const formatDateTimeString = (
   dateTimeString: string | number,
   useTimeZone = false,
 ) => {
-  // eslint-disable-line arrow-body-style
   return `${formatDateString(dateTimeString, useTimeZone)} ${formatTimeString(
     dateTimeString,
     useTimeZone,

package/src/intervalUtils.ts

@@ -52,7 +52,6 @@ const extractDurationFromRegexMatch = (
   negative: boolean,
   match: RegExpExecArray,
 ): Duration => {
-  // eslint-disable-next-line no-magic-numbers
   const milliseconds = parseInt(match[4]?.padEnd(3, "0") ?? "0", 10);
 
   return {
@@ -104,7 +103,6 @@ export const formatIntervalToISOIntervalString = (
   } else {
     output += `T${toFormat.interval.hours}H${toFormat.interval.minutes}M${toFormat.interval.seconds}.${toFormat.interval.milliseconds
       .toString()
-      // eslint-disable-next-line no-magic-numbers
       .padStart(3, "0")}S`;
   }
 
@@ -308,7 +306,7 @@ export const isValidHumanReadableIntervalString = (
     } else {
       return { valid: false };
     }
-  } catch (e) {
+  } catch (_e) {
     return { valid: false };
   }
 };
@@ -384,7 +382,7 @@ export const isValidISOIntervalString = (
     } else {
       return { valid: false };
     }
-  } catch (e) {
+  } catch (_e) {
     return { valid: false };
   }
 };

package/src/uploadManager/index.ts

@@ -1,6 +1,5 @@
 import { AbortError, createAbortablePromise, retryPromise } from "../promise";
 
-// eslint-disable-next-line no-magic-numbers
 const CHUNK_SIZE = 50 * 1024 * 1024; // 50 MB per chunk
 
 const DEFAULT_RETRY_COUNT = 5;

package/test-utils/muteConsole.ts

@@ -8,6 +8,7 @@ const methods: ConsoleMethod[] = ["log", "warn", "error", "info"];
  * @param f The function to run. Can be async.
  * @returns The original function's return value
  */
+// TODO: replace Function with fitting top type (...args: never[]) => unknown has unknown return type which makes problems in line 41
 export default (f: Function) => {
   const originals = methods.map((method) => console[method]);
   methods.forEach((method) => {
@@ -15,11 +16,11 @@ export default (f: Function) => {
   });
 
   let originalConsola: typeof consola;
-  // @ts-ignore - needs to extend the globalThis namespace
+  // @ts-expect-error Property 'consola' does not exist on type 'typeof globalThis'.
   if (global.consola) {
-    // @ts-ignore
+    // @ts-expect-error Property 'consola' does not exist on type 'typeof globalThis'.
     originalConsola = global.consola;
-    // @ts-ignore
+    // @ts-expect-error Property 'consola' does not exist on type 'typeof globalThis'.
     global.consola = consola.create({
       level: -1,
     });
@@ -30,7 +31,7 @@ export default (f: Function) => {
       console[method] = originals[i];
     });
     if (originalConsola) {
-      // @ts-ignore
+      // @ts-expect-error Property 'consola' does not exist on type 'typeof globalThis'.
       global.consola = originalConsola;
     }
   };