From 42ccabf3b5a2d8f1e071e5572199e13f0922b9f6 Mon Sep 17 00:00:00 2001 From: chad1008 <13856531+chad1008@users.noreply.github.com> Date: Mon, 17 Oct 2022 15:11:34 -0400 Subject: [PATCH] also ignore useUpdateEffect from utils directory --- packages/components/src/utils/hooks/use-update-effect.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/packages/components/src/utils/hooks/use-update-effect.js b/packages/components/src/utils/hooks/use-update-effect.js index aaf04478b3860d..f20c71e1b56f3a 100644 --- a/packages/components/src/utils/hooks/use-update-effect.js +++ b/packages/components/src/utils/hooks/use-update-effect.js @@ -20,6 +20,11 @@ function useUpdateEffect( effect, deps ) { } mounted.current = true; return undefined; + // Disable reasons: + // 1. This hook needs to pass a dep list that isn't an array literal + // 2. `effect` is missing from the array, and will need to be added carefully to avoid additional warnings + // see https://github.com/WordPress/gutenberg/pull/41166 + // eslint-disable-next-line react-hooks/exhaustive-deps }, deps ); }