From 818f57dd639b9de7d70d5b269c6ab22491bfe662 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E0=A4=95=E0=A4=BE=E0=A4=B0=E0=A4=A4=E0=A5=8B=E0=A4=AB?= =?UTF-8?q?=E0=A5=8D=E0=A4=AB=E0=A5=87=E0=A4=B2=E0=A4=B8=E0=A5=8D=E0=A4=95?= =?UTF-8?q?=E0=A5=8D=E0=A4=B0=E0=A4=BF=E0=A4=AA=E0=A5=8D=E0=A4=9F=E2=84=A2?= Date: Fri, 28 Oct 2022 16:10:56 +0200 Subject: [PATCH] refactor: remove console.warn from `deepCopy` (#4471) Jan realized that this was logging way too much (as in, we have way too many circular references). I've changed this to be reported to Sentry, which should hopefully get merged next week. --- packages/workflow/src/utils.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/packages/workflow/src/utils.ts b/packages/workflow/src/utils.ts index ddd9e1412..87fa050a5 100644 --- a/packages/workflow/src/utils.ts +++ b/packages/workflow/src/utils.ts @@ -8,7 +8,6 @@ export const deepCopy = (source: T, hash = new WeakMap(), path = ''): T => { return source; } if (hash.has(source)) { - console.warn(`Circular reference detected at "source${path}"`); return hash.get(source); } // Date