fixup! Feat(common): Introduce info and warning utilities

lmc-eu · Jan 5, 2024 · 3f4b632 · 3f4b632
1 parent 55ff16b
commit 3f4b632
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/packages/common/utilities/__tests__/warning.test.ts b/packages/common/utilities/__tests__/warning.test.ts
@@ -1,5 +1,6 @@
-// we are testing Console function
-/* eslint-disable no-console */
+/* eslint-disable no-console --
+ * we are testing Console function
+ */
 import warning from '../warning';
 
 describe('warning', () => {

diff --git a/packages/common/utilities/info.ts b/packages/common/utilities/info.ts
@@ -12,11 +12,8 @@ const info = (condition: unknown, message: string): void => {
     // Condition not passed
     const text = `Info: ${message}`;
 
-    // check console for IE9 support which provides console
-    // only with open devtools
     if (typeof console !== 'undefined') {
-      // we want to log a warning; so usage of Console is required
-      // eslint-disable-next-line no-console
+      // eslint-disable-next-line no-console -- we want to log a warning; so usage of Console is required
       console.info(text);
     }
   }

diff --git a/packages/common/utilities/warning.ts b/packages/common/utilities/warning.ts
@@ -12,8 +12,6 @@ const warning = (condition: unknown, message: string): void => {
     // Condition not passed
     const text = `Warning: ${message}`;
 
-    // check console for IE9 support which provides console
-    // only with open devtools
     if (typeof console !== 'undefined') {
       // we want to log a warning; so usage of Console is required
       // eslint-disable-next-line no-console