Just remove @deephaven/log from golden-layout

- Couldn't get the ignore to work, not interested in rewiring it right now - The event still gets logged with the LogProxy and still appears in the exported support logs.
deephaven · Jun 8, 2023 · 37f61f2 · 37f61f2
1 parent 9acb901
commit 37f61f2
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 11 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/golden-layout/package.json b/packages/golden-layout/package.json
@@ -11,7 +11,6 @@
   "type": "module",
   "dependencies": {
     "@deephaven/components": "file:../components",
-    "@deephaven/log": "file:../log",
     "jquery": "^3.6.0"
   },
   "peerDependencies": {

diff --git a/packages/golden-layout/src/utils/EventEmitter.ts b/packages/golden-layout/src/utils/EventEmitter.ts
@@ -1,7 +1,3 @@
-import Log from '@deephaven/log';
-
-const log = Log.module('EventEmitter');
-
 /**
  * A generic and very fast EventEmitter
  * implementation. On top of emitting the
@@ -72,7 +68,7 @@ class EventEmitter {
         try {
           subs[i].fn.apply(ctx, args);
         } catch (e) {
-          log.error('Error while emitting event:', e);
+          console.error('Error while emitting event:', e);
         }
       }
     }
@@ -86,7 +82,7 @@ class EventEmitter {
       try {
         allEventSubs[i].fn.apply(ctx, args);
       } catch (e) {
-        log.error('Error while emitting event to allEventSubs:', e);
+        console.error('Error while emitting event to allEventSubs:', e);
       }
     }
   }

diff --git a/packages/golden-layout/tsconfig.json b/packages/golden-layout/tsconfig.json
@@ -5,6 +5,5 @@
     "outDir": "dist/"
   },
   "include": ["src/**/*.ts", "src/**/*.tsx", "src/**/*.js", "src/**/*.jsx"],
-  "exclude": ["node_modules"],
-  "references": [{ "path": "../components" }, { "path": "../log" }]
+  "exclude": ["node_modules"]
 }