From cf44858dc193941b068b15797ceb64a030f3d914 Mon Sep 17 00:00:00 2001
From: Aaron Shafovaloff <ashafovaloff@instructure.com>
Date: Fri, 21 Apr 2023 13:55:03 -0600
Subject: [PATCH] turn variables off in no-use-before-define rule

When set to false, "the rule ignores a reference if the declaration
is in an upper scope, while still reporting the reference if it's
in the same scope as the declaration."
https://eslint.org/docs/latest/rules/no-use-before-define

test plan:
  - existing tests pass

flag=none

Change-Id: I3b4d071d97e713f4f02842ac9a684080340e71f8
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/316676
Reviewed-by: Kai Bjorkman <kbjorkman@instructure.com>
Reviewed-by: Cameron Ray <cameron.ray@instructure.com>
QA-Review: Kai Bjorkman <kbjorkman@instructure.com>
Product-Review: Cameron Ray <cameron.ray@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
---
 .eslintrc.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.eslintrc.js b/.eslintrc.js
index 6e4eb0f0881..d40fe52819c 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -96,6 +96,7 @@ module.exports = {
       {
         functions: false,
         classes: false,
+        variables: false,
       },
     ],
     'no-useless-escape': 'off',