From 0086bf28084e6ab53b5ca30fb11c00aaa198a90c Mon Sep 17 00:00:00 2001
From: Jan Kassens <jkassens@meta.com>
Date: Mon, 19 Dec 2022 15:13:53 -0500
Subject: [PATCH] lint: enable reportUnusedDisableDirectives and remove unused
 suppressions

This enables linting against unused suppressions and removes the ones that were unused.
---
 .eslintrc.js                                  |  2 ++
 .../src/ReactFlightReplyClient.js             |  4 ---
 .../forks/ReactFlightClientConfig.custom.js   |  2 +-
 .../flow-typed/jest.js                        |  5 ---
 .../npm/react-test-renderer_v16.x.x.js        |  1 -
 .../src/__tests__/inspectedElement-test.js    |  3 +-
 .../__tests__/legacy/inspectElement-test.js   |  3 +-
 .../src/backend/DevToolsConsolePatching.js    |  2 --
 .../src/backend/renderer.js                   |  1 -
 .../UnserializableProps.js                    |  2 +-
 packages/react-devtools/app.js                |  2 +-
 .../src/client/DOMPropertyOperations.js       |  4 +--
 .../src/client/ReactFiberConfigDOM.js         |  2 +-
 .../src/client/validateDOMNesting.js          |  1 -
 .../src/server/ReactFizzConfigDOM.js          |  2 +-
 .../src/shared/ReactDOMUnknownPropertyHook.js |  2 +-
 .../src/shared/isAttributeNameSafe.js         |  2 --
 .../src/shared/sanitizeURL.js                 |  1 -
 .../src/__tests__/ReactComponent-test.js      |  2 --
 .../__tests__/ReactComponentLifeCycle-test.js |  2 --
 .../ReactDOMServerIntegrationElements-test.js |  2 +-
 ...ctDOMServerIntegrationReconnecting-test.js |  6 ++--
 .../src/__tests__/ReactLegacyUpdates-test.js  |  2 --
 .../src/__tests__/ReactUpdates-test.js        |  2 --
 packages/react-dom/src/client/ReactDOM.js     |  2 --
 packages/react-native-renderer/index.js       |  1 -
 .../src/createReactNoop.js                    |  1 -
 packages/react-reconciler/src/ReactFiber.js   |  4 +--
 .../src/ReactFiberClassComponent.js           |  2 +-
 .../src/ReactFiberCompleteWork.js             |  1 -
 .../src/ReactFiberWorkLoop.js                 |  3 --
 .../src/__tests__/ReactEffectOrdering-test.js |  2 --
 .../src/__tests__/ReactHooks-test.internal.js | 10 ------
 .../src/__tests__/useEffectEvent-test.js      |  2 --
 .../src/__tests__/useRef-test.internal.js     |  2 --
 .../src/forks/ReactFiberConfig.custom.js      | 32 +++++++++----------
 .../src/__tests__/ReactFresh-test.js          |  2 --
 .../__tests__/ReactFreshIntegration-test.js   |  2 --
 .../src/ReactFlightDOMServerNode.js           |  1 -
 .../ReactFlightServerTemporaryReferences.js   |  1 -
 .../src/forks/ReactFizzConfig.custom.js       |  2 +-
 .../forks/ReactServerStreamConfig.custom.js   |  8 ++---
 .../src/ReactTestRenderer.js                  |  4 +--
 .../src/__tests__/ReactStrictMode-test.js     |  6 ----
 .../npm/umd/scheduler.development.js          |  2 --
 .../npm/umd/scheduler.production.min.js       |  2 --
 .../npm/umd/scheduler.profiling.min.js        |  2 --
 .../scheduler/src/__tests__/Scheduler-test.js |  2 --
 .../__tests__/SchedulerSetImmediate-test.js   |  2 --
 packages/shared/ConsolePatchingDev.js         |  3 --
 packages/shared/isArray.js                    |  1 -
 packages/shared/reportGlobalError.js          |  1 -
 .../babel/transform-react-version-pragma.js   |  2 --
 .../__tests__/transform-error-messages.js     |  2 +-
 scripts/flow/react-native-host-hooks.js       |  4 +--
 scripts/jest/devtools/setupEnv.js             |  1 -
 56 files changed, 44 insertions(+), 125 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 005727b5e14d4..b92a0c353a983 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -17,6 +17,8 @@ module.exports = {
   // Stop ESLint from looking for a configuration file in parent folders
   root: true,
 
+  reportUnusedDisableDirectives: true,
+
   plugins: [
     'babel',
     'ft-flow',
diff --git a/packages/react-client/src/ReactFlightReplyClient.js b/packages/react-client/src/ReactFlightReplyClient.js
index e3d19319c2f45..cc2c3e3482f39 100644
--- a/packages/react-client/src/ReactFlightReplyClient.js
+++ b/packages/react-client/src/ReactFlightReplyClient.js
@@ -243,7 +243,6 @@ export function processReply(
             const partJSON = JSON.stringify(resolvedModel, resolveToJSON);
             // $FlowFixMe[incompatible-type] We know it's not null because we assigned it above.
             const data: FormData = formData;
-            // eslint-disable-next-line react-internal/safe-string-coercion
             data.append(formFieldPrefix + lazyId, partJSON);
             return serializeByValueID(lazyId);
           } catch (x) {
@@ -263,7 +262,6 @@ export function processReply(
                   const partJSON = JSON.stringify(value, resolveToJSON);
                   // $FlowFixMe[incompatible-type] We know it's not null because we assigned it above.
                   const data: FormData = formData;
-                  // eslint-disable-next-line react-internal/safe-string-coercion
                   data.append(formFieldPrefix + lazyId, partJSON);
                   pendingParts--;
                   if (pendingParts === 0) {
@@ -304,7 +302,6 @@ export function processReply(
               const partJSON = JSON.stringify(partValue, resolveToJSON);
               // $FlowFixMe[incompatible-type] We know it's not null because we assigned it above.
               const data: FormData = formData;
-              // eslint-disable-next-line react-internal/safe-string-coercion
               data.append(formFieldPrefix + promiseId, partJSON);
               pendingParts--;
               if (pendingParts === 0) {
@@ -459,7 +456,6 @@ export function processReply(
         }
         // The reference to this function came from the same client so we can pass it back.
         const refId = nextPartId++;
-        // eslint-disable-next-line react-internal/safe-string-coercion
         formData.set(formFieldPrefix + refId, metaDataJSON);
         return serializeServerReferenceID(refId);
       }
diff --git a/packages/react-client/src/forks/ReactFlightClientConfig.custom.js b/packages/react-client/src/forks/ReactFlightClientConfig.custom.js
index 3add5988aab8a..88694f59324d4 100644
--- a/packages/react-client/src/forks/ReactFlightClientConfig.custom.js
+++ b/packages/react-client/src/forks/ReactFlightClientConfig.custom.js
@@ -42,7 +42,7 @@ export const usedWithSSR = true;
 
 export opaque type Source = mixed;
 
-export opaque type StringDecoder = mixed; // eslint-disable-line no-undef
+export opaque type StringDecoder = mixed;
 
 export const createStringDecoder = $$$config.createStringDecoder;
 export const readPartialStringChunk = $$$config.readPartialStringChunk;
diff --git a/packages/react-devtools-extensions/flow-typed/jest.js b/packages/react-devtools-extensions/flow-typed/jest.js
index cdbe32bee7ba6..66c8f0d41fe55 100644
--- a/packages/react-devtools-extensions/flow-typed/jest.js
+++ b/packages/react-devtools-extensions/flow-typed/jest.js
@@ -146,13 +146,11 @@ type JestPromiseType = {
    * Use rejects to unwrap the reason of a rejected promise so any other
    * matcher can be chained. If the promise is fulfilled the assertion fails.
    */
-  // eslint-disable-next-line no-use-before-define
   rejects: JestExpectType,
   /**
    * Use resolves to unwrap the value of a fulfilled promise so any other
    * matcher can be chained. If the promise is rejected the assertion fails.
    */
-  // eslint-disable-next-line no-use-before-define
   resolves: JestExpectType,
 };
 
@@ -1098,10 +1096,8 @@ type JestPrettyFormatColors = {
 };
 
 type JestPrettyFormatIndent = string => string;
-// eslint-disable-next-line no-unused-vars
 type JestPrettyFormatRefs = Array<any>;
 type JestPrettyFormatPrint = any => string;
-// eslint-disable-next-line no-unused-vars
 type JestPrettyFormatStringOrNull = string | null;
 
 type JestPrettyFormatOptions = {
@@ -1112,7 +1108,6 @@ type JestPrettyFormatOptions = {
   indent: number,
   maxDepth: number,
   min: boolean,
-  // eslint-disable-next-line no-use-before-define
   plugins: JestPrettyFormatPlugins,
   printFunctionName: boolean,
   spacing: string,
diff --git a/packages/react-devtools-extensions/flow-typed/npm/react-test-renderer_v16.x.x.js b/packages/react-devtools-extensions/flow-typed/npm/react-test-renderer_v16.x.x.js
index f706d830ebc00..c9a96b1d4b9f8 100644
--- a/packages/react-devtools-extensions/flow-typed/npm/react-test-renderer_v16.x.x.js
+++ b/packages/react-devtools-extensions/flow-typed/npm/react-test-renderer_v16.x.x.js
@@ -52,7 +52,6 @@ type TestRendererOptions = {
 };
 
 declare module 'react-test-renderer' {
-  // eslint-disable-next-line no-inner-declarations
   declare export type ReactTestRenderer = {
     toJSON(): null | ReactTestRendererJSON,
     toTree(): null | ReactTestRendererTree,
diff --git a/packages/react-devtools-shared/src/__tests__/inspectedElement-test.js b/packages/react-devtools-shared/src/__tests__/inspectedElement-test.js
index f32971a1d814f..5db7a1378cbbf 100644
--- a/packages/react-devtools-shared/src/__tests__/inspectedElement-test.js
+++ b/packages/react-devtools-shared/src/__tests__/inspectedElement-test.js
@@ -664,7 +664,6 @@ describe('InspectedElement', () => {
           anonymous_fn={instance.anonymousFunction}
           array_buffer={arrayBuffer}
           array_of_arrays={arrayOfArrays}
-          // eslint-disable-next-line no-undef
           big_int={BigInt(123)}
           bound_fn={exampleFunction.bind(this)}
           data_view={dataView}
@@ -1874,7 +1873,7 @@ describe('InspectedElement', () => {
         xyz: 1,
       },
     });
-    const bigInt = BigInt(123); // eslint-disable-line no-undef
+    const bigInt = BigInt(123);
 
     await utils.actAsync(() =>
       render(
diff --git a/packages/react-devtools-shared/src/__tests__/legacy/inspectElement-test.js b/packages/react-devtools-shared/src/__tests__/legacy/inspectElement-test.js
index 1940e82a80f40..ebefdc774c0c3 100644
--- a/packages/react-devtools-shared/src/__tests__/legacy/inspectElement-test.js
+++ b/packages/react-devtools-shared/src/__tests__/legacy/inspectElement-test.js
@@ -182,7 +182,6 @@ describe('InspectedElementContext', () => {
           anonymous_fn={instance.anonymousFunction}
           array_buffer={arrayBuffer}
           array_of_arrays={arrayOfArrays}
-          // eslint-disable-next-line no-undef
           big_int={BigInt(123)}
           bound_fn={exampleFunction.bind(this)}
           data_view={dataView}
@@ -868,7 +867,7 @@ describe('InspectedElementContext', () => {
         xyz: 1,
       },
     });
-    const bigInt = BigInt(123); // eslint-disable-line no-undef
+    const bigInt = BigInt(123);
 
     act(() =>
       ReactDOM.render(
diff --git a/packages/react-devtools-shared/src/backend/DevToolsConsolePatching.js b/packages/react-devtools-shared/src/backend/DevToolsConsolePatching.js
index bc133590f89b8..71ede689250ea 100644
--- a/packages/react-devtools-shared/src/backend/DevToolsConsolePatching.js
+++ b/packages/react-devtools-shared/src/backend/DevToolsConsolePatching.js
@@ -30,7 +30,6 @@ disabledLog.__reactDisabledLog = true;
 
 export function disableLogs(): void {
   if (disabledDepth === 0) {
-    /* eslint-disable react-internal/no-production-logging */
     prevLog = console.log;
     prevInfo = console.info;
     prevWarn = console.warn;
@@ -63,7 +62,6 @@ export function disableLogs(): void {
 export function reenableLogs(): void {
   disabledDepth--;
   if (disabledDepth === 0) {
-    /* eslint-disable react-internal/no-production-logging */
     const props = {
       configurable: true,
       enumerable: true,
diff --git a/packages/react-devtools-shared/src/backend/renderer.js b/packages/react-devtools-shared/src/backend/renderer.js
index b1e5dac527e61..4b79108617f4a 100644
--- a/packages/react-devtools-shared/src/backend/renderer.js
+++ b/packages/react-devtools-shared/src/backend/renderer.js
@@ -1545,7 +1545,6 @@ export function attach(
     return changedKeys;
   }
 
-  // eslint-disable-next-line no-unused-vars
   function didFiberRender(prevFiber: Fiber, nextFiber: Fiber): boolean {
     switch (nextFiber.tag) {
       case ClassComponent:
diff --git a/packages/react-devtools-shell/src/app/InspectableElements/UnserializableProps.js b/packages/react-devtools-shell/src/app/InspectableElements/UnserializableProps.js
index 906683c0975fb..2c279a1bdbf87 100644
--- a/packages/react-devtools-shell/src/app/InspectableElements/UnserializableProps.js
+++ b/packages/react-devtools-shell/src/app/InspectableElements/UnserializableProps.js
@@ -31,7 +31,7 @@ const immutable = Immutable.fromJS({
     xyz: 1,
   },
 });
-const bigInt = BigInt(123); // eslint-disable-line no-undef
+const bigInt = BigInt(123);
 
 class Foo {
   flag = false;
diff --git a/packages/react-devtools/app.js b/packages/react-devtools/app.js
index 3912a80ef09d5..c427852b408ba 100644
--- a/packages/react-devtools/app.js
+++ b/packages/react-devtools/app.js
@@ -46,7 +46,7 @@ app.on('ready', function () {
   });
 
   // and load the index.html of the app.
-  mainWindow.loadURL('file://' + __dirname + '/app.html'); // eslint-disable-line no-path-concat
+  mainWindow.loadURL('file://' + __dirname + '/app.html');
   // $FlowFixMe[incompatible-use] found when upgrading Flow
   mainWindow.webContents.executeJavaScript(
     // We use this so that RN can keep relative JSX __source filenames
diff --git a/packages/react-dom-bindings/src/client/DOMPropertyOperations.js b/packages/react-dom-bindings/src/client/DOMPropertyOperations.js
index 867eeba4a0869..1d5211e451ff2 100644
--- a/packages/react-dom-bindings/src/client/DOMPropertyOperations.js
+++ b/packages/react-dom-bindings/src/client/DOMPropertyOperations.js
@@ -30,7 +30,7 @@ export function getValueForAttribute(
       // shouldRemoveAttribute
       switch (typeof expected) {
         case 'function':
-        case 'symbol': // eslint-disable-line
+        case 'symbol':
           return expected;
         case 'boolean': {
           const prefix = name.toLowerCase().slice(0, 5);
@@ -109,7 +109,7 @@ export function setValueForAttribute(
     switch (typeof value) {
       case 'undefined':
       case 'function':
-      case 'symbol': // eslint-disable-line
+      case 'symbol':
         node.removeAttribute(name);
         return;
       case 'boolean': {
diff --git a/packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js b/packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js
index d67571fc41e1d..196c82b1bc9f1 100644
--- a/packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js
+++ b/packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js
@@ -441,7 +441,7 @@ export function createInstance(
               didWarnScriptTags = true;
             }
           }
-          div.innerHTML = '<script><' + '/script>'; // eslint-disable-line
+          div.innerHTML = '<script><' + '/script>';
           // This is guaranteed to yield a script element.
           const firstChild = ((div.firstChild: any): HTMLScriptElement);
           domElement = div.removeChild(firstChild);
diff --git a/packages/react-dom-bindings/src/client/validateDOMNesting.js b/packages/react-dom-bindings/src/client/validateDOMNesting.js
index ab49cb4f4b025..2048aafdb67cc 100644
--- a/packages/react-dom-bindings/src/client/validateDOMNesting.js
+++ b/packages/react-dom-bindings/src/client/validateDOMNesting.js
@@ -502,7 +502,6 @@ function validateTextNesting(childText: string, parentTag: string): boolean {
       return true;
     }
 
-    // eslint-disable-next-line react-internal/safe-string-coercion
     const warnKey = '#text|' + parentTag;
     if (didWarn[warnKey]) {
       return false;
diff --git a/packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js b/packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js
index 623bedd92e6e0..6d1382ee4abaa 100644
--- a/packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js
+++ b/packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js
@@ -1452,7 +1452,7 @@ function pushAttribute(
         // shouldRemoveAttribute
         switch (typeof value) {
           case 'function':
-          case 'symbol': // eslint-disable-line
+          case 'symbol':
             return;
           case 'boolean': {
             const prefix = attributeName.toLowerCase().slice(0, 5);
diff --git a/packages/react-dom-bindings/src/shared/ReactDOMUnknownPropertyHook.js b/packages/react-dom-bindings/src/shared/ReactDOMUnknownPropertyHook.js
index 3e49fa637f29d..f45e477d876f7 100644
--- a/packages/react-dom-bindings/src/shared/ReactDOMUnknownPropertyHook.js
+++ b/packages/react-dom-bindings/src/shared/ReactDOMUnknownPropertyHook.js
@@ -271,7 +271,7 @@ function validateProperty(tagName, name, value, eventRegistry) {
         }
       }
       case 'function':
-      case 'symbol': // eslint-disable-line
+      case 'symbol':
         // Warn when a known attribute is a bad type
         warnedProperties[name] = true;
         return false;
diff --git a/packages/react-dom-bindings/src/shared/isAttributeNameSafe.js b/packages/react-dom-bindings/src/shared/isAttributeNameSafe.js
index d1378a64de37c..77cf005e404e7 100644
--- a/packages/react-dom-bindings/src/shared/isAttributeNameSafe.js
+++ b/packages/react-dom-bindings/src/shared/isAttributeNameSafe.js
@@ -9,10 +9,8 @@
 
 import hasOwnProperty from 'shared/hasOwnProperty';
 
-/* eslint-disable max-len */
 const ATTRIBUTE_NAME_START_CHAR =
   ':A-Z_a-z\\u00C0-\\u00D6\\u00D8-\\u00F6\\u00F8-\\u02FF\\u0370-\\u037D\\u037F-\\u1FFF\\u200C-\\u200D\\u2070-\\u218F\\u2C00-\\u2FEF\\u3001-\\uD7FF\\uF900-\\uFDCF\\uFDF0-\\uFFFD';
-/* eslint-enable max-len */
 export const ATTRIBUTE_NAME_CHAR: string =
   ATTRIBUTE_NAME_START_CHAR + '\\-.0-9\\u00B7\\u0300-\\u036F\\u203F-\\u2040';
 
diff --git a/packages/react-dom-bindings/src/shared/sanitizeURL.js b/packages/react-dom-bindings/src/shared/sanitizeURL.js
index d1f0e62cbac76..477bb9f9c921e 100644
--- a/packages/react-dom-bindings/src/shared/sanitizeURL.js
+++ b/packages/react-dom-bindings/src/shared/sanitizeURL.js
@@ -16,7 +16,6 @@
 // INFORMATION SEPARATOR ONE, inclusive:
 // https://infra.spec.whatwg.org/#c0-control-or-space
 
-/* eslint-disable max-len */
 const isJavaScriptProtocol =
   /^[\u0000-\u001F ]*j[\r\n\t]*a[\r\n\t]*v[\r\n\t]*a[\r\n\t]*s[\r\n\t]*c[\r\n\t]*r[\r\n\t]*i[\r\n\t]*p[\r\n\t]*t[\r\n\t]*\:/i;
 
diff --git a/packages/react-dom/src/__tests__/ReactComponent-test.js b/packages/react-dom/src/__tests__/ReactComponent-test.js
index 1a77cbad680af..c697d8a45bbfe 100644
--- a/packages/react-dom/src/__tests__/ReactComponent-test.js
+++ b/packages/react-dom/src/__tests__/ReactComponent-test.js
@@ -424,7 +424,6 @@ describe('ReactComponent', () => {
       root.unmount();
     });
 
-    /* eslint-disable indent */
     expect(log).toEqual([
       'start mount',
       'inner 1 render',
@@ -453,7 +452,6 @@ describe('ReactComponent', () => {
       'ref 2 got null',
       'inner 2 componentWillUnmount',
     ]);
-    /* eslint-enable indent */
   });
 
   // @gate !disableLegacyMode
diff --git a/packages/react-dom/src/__tests__/ReactComponentLifeCycle-test.js b/packages/react-dom/src/__tests__/ReactComponentLifeCycle-test.js
index f89e95bcf5dde..b8f6858a76d80 100644
--- a/packages/react-dom/src/__tests__/ReactComponentLifeCycle-test.js
+++ b/packages/react-dom/src/__tests__/ReactComponentLifeCycle-test.js
@@ -1505,7 +1505,6 @@ describe('ReactComponentLifeCycle', () => {
       });
     }).toWarnDev(
       [
-        /* eslint-disable max-len */
         `Warning: componentWillMount has been renamed, and is not recommended for use. See https://react.dev/link/unsafe-component-lifecycles for details.
 
 * Move code with side effects to componentDidMount, and set initial state in the constructor.
@@ -1525,7 +1524,6 @@ Please update the following components: MyComponent`,
 * Rename componentWillUpdate to UNSAFE_componentWillUpdate to suppress this warning in non-strict mode. In React 18.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run \`npx react-codemod rename-unsafe-lifecycles\` in your project source folder.
 
 Please update the following components: MyComponent`,
-        /* eslint-enable max-len */
       ],
       {withoutStack: true},
     );
diff --git a/packages/react-dom/src/__tests__/ReactDOMServerIntegrationElements-test.js b/packages/react-dom/src/__tests__/ReactDOMServerIntegrationElements-test.js
index f492aebb455db..114f4fb447751 100644
--- a/packages/react-dom/src/__tests__/ReactDOMServerIntegrationElements-test.js
+++ b/packages/react-dom/src/__tests__/ReactDOMServerIntegrationElements-test.js
@@ -744,7 +744,7 @@ describe('ReactDOMServerIntegration', () => {
         'a div with a single child surrounded by whitespace',
         async render => {
           // prettier-ignore
-          const e = await render(<div id="parent">  <div id="child" />   </div>); // eslint-disable-line no-multi-spaces
+          const e = await render(<div id="parent">  <div id="child" />   </div>);
           expect(e.childNodes.length).toBe(3);
           const textNode1 = e.childNodes[0];
           const child = e.childNodes[1];
diff --git a/packages/react-dom/src/__tests__/ReactDOMServerIntegrationReconnecting-test.js b/packages/react-dom/src/__tests__/ReactDOMServerIntegrationReconnecting-test.js
index a58f7f8529f98..3466b67258959 100644
--- a/packages/react-dom/src/__tests__/ReactDOMServerIntegrationReconnecting-test.js
+++ b/packages/react-dom/src/__tests__/ReactDOMServerIntegrationReconnecting-test.js
@@ -310,13 +310,13 @@ describe('ReactDOMServerIntegration', () => {
             <div id="child2" />
           </div>,
           // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
         ));
 
       it('should error reconnecting a div with children separated by different whitespace on the server', () =>
         expectMarkupMismatch(
           // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
           <div id="parent">
             <div id="child1" />
             <div id="child2" />
@@ -329,7 +329,7 @@ describe('ReactDOMServerIntegration', () => {
             <div id="child1" /> <div id="child2" />
           </div>,
           // prettier-ignore
-          <div id="parent"><div id="child1" />      <div id="child2" /></div>, // eslint-disable-line no-multi-spaces
+          <div id="parent"><div id="child1" />      <div id="child2" /></div>,
         ));
 
       it('can distinguish an empty component from a dom node', () =>
diff --git a/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js b/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js
index 942191372a3c0..e42e8e3b78d35 100644
--- a/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js
+++ b/packages/react-dom/src/__tests__/ReactLegacyUpdates-test.js
@@ -651,7 +651,6 @@ describe('ReactLegacyUpdates', () => {
       });
     });
 
-    /* eslint-disable indent */
     expect(updates).toEqual([
       'Outer-render-0',
       'Inner-render-0-0',
@@ -680,7 +679,6 @@ describe('ReactLegacyUpdates', () => {
       'Inner-didUpdate-2-2',
       'Inner-callback-2',
     ]);
-    /* eslint-enable indent */
   });
 
   // @gate !disableLegacyMode
diff --git a/packages/react-dom/src/__tests__/ReactUpdates-test.js b/packages/react-dom/src/__tests__/ReactUpdates-test.js
index d4acfaa9b35f8..cb6a06c0cb610 100644
--- a/packages/react-dom/src/__tests__/ReactUpdates-test.js
+++ b/packages/react-dom/src/__tests__/ReactUpdates-test.js
@@ -759,7 +759,6 @@ describe('ReactUpdates', () => {
       });
     });
 
-    /* eslint-disable indent */
     expect(updates).toEqual([
       'Outer-render-0',
       'Inner-render-0-0',
@@ -788,7 +787,6 @@ describe('ReactUpdates', () => {
       'Inner-didUpdate-2-2',
       'Inner-callback-2',
     ]);
-    /* eslint-enable indent */
   });
 
   it('should flush updates in the correct order across roots', async () => {
diff --git a/packages/react-dom/src/client/ReactDOM.js b/packages/react-dom/src/client/ReactDOM.js
index 99d2b62f2f845..49c2b53fa4673 100644
--- a/packages/react-dom/src/client/ReactDOM.js
+++ b/packages/react-dom/src/client/ReactDOM.js
@@ -147,9 +147,7 @@ function hydrateRoot(
 // Overload the definition to the two valid signatures.
 // Warning, this opts-out of checking the function body.
 declare function flushSync<R>(fn: () => R): R;
-// eslint-disable-next-line no-redeclare
 declare function flushSync(): void;
-// eslint-disable-next-line no-redeclare
 function flushSync<R>(fn: (() => R) | void): R | void {
   if (__DEV__) {
     if (isAlreadyRendering()) {
diff --git a/packages/react-native-renderer/index.js b/packages/react-native-renderer/index.js
index da522f358f21d..ecc5e58c3d8d6 100644
--- a/packages/react-native-renderer/index.js
+++ b/packages/react-native-renderer/index.js
@@ -10,7 +10,6 @@
 import type {ReactNativeType} from './src/ReactNativeTypes';
 import * as ReactNative from './src/ReactNativeRenderer';
 // Assert that the exports line up with the type we're going to expose.
-// eslint-disable-next-line ft-flow/no-unused-expressions
 (ReactNative: ReactNativeType);
 
 export * from './src/ReactNativeRenderer';
diff --git a/packages/react-noop-renderer/src/createReactNoop.js b/packages/react-noop-renderer/src/createReactNoop.js
index af4612f00ddee..ab6b8ede43a26 100644
--- a/packages/react-noop-renderer/src/createReactNoop.js
+++ b/packages/react-noop-renderer/src/createReactNoop.js
@@ -919,7 +919,6 @@ function createReactNoop(reconciler: Function, useMutation: boolean) {
 
   function onRecoverableError(error) {
     // TODO: Turn this on once tests are fixed
-    // eslint-disable-next-line react-internal/no-production-logging, react-internal/warning-args
     // console.error(error);
   }
 
diff --git a/packages/react-reconciler/src/ReactFiber.js b/packages/react-reconciler/src/ReactFiber.js
index 5b8dc06fb5f69..95b53840831c9 100644
--- a/packages/react-reconciler/src/ReactFiber.js
+++ b/packages/react-reconciler/src/ReactFiber.js
@@ -119,10 +119,10 @@ if (__DEV__) {
   hasBadMapPolyfill = false;
   try {
     const nonExtensibleObject = Object.preventExtensions({});
-    /* eslint-disable no-new */
+    // eslint-disable-next-line no-new
     new Map([[nonExtensibleObject, null]]);
+    // eslint-disable-next-line no-new
     new Set([nonExtensibleObject]);
-    /* eslint-enable no-new */
   } catch (e) {
     // TODO: Consider warning about bad polyfills
     hasBadMapPolyfill = true;
diff --git a/packages/react-reconciler/src/ReactFiberClassComponent.js b/packages/react-reconciler/src/ReactFiberClassComponent.js
index b64f9f9e626c8..855af65553756 100644
--- a/packages/react-reconciler/src/ReactFiberClassComponent.js
+++ b/packages/react-reconciler/src/ReactFiberClassComponent.js
@@ -648,7 +648,7 @@ function constructClassInstance(
     ) {
       setIsStrictModeForDevtools(true);
       try {
-        instance = new ctor(props, context); // eslint-disable-line no-new
+        instance = new ctor(props, context);
       } finally {
         setIsStrictModeForDevtools(false);
       }
diff --git a/packages/react-reconciler/src/ReactFiberCompleteWork.js b/packages/react-reconciler/src/ReactFiberCompleteWork.js
index 89044182672ad..ee4bd627d0f70 100644
--- a/packages/react-reconciler/src/ReactFiberCompleteWork.js
+++ b/packages/react-reconciler/src/ReactFiberCompleteWork.js
@@ -328,7 +328,6 @@ function appendAllChildrenToContainer(
     // children to find all the terminal nodes.
     let node = workInProgress.child;
     while (node !== null) {
-      // eslint-disable-next-line no-labels
       if (node.tag === HostComponent) {
         let instance = node.stateNode;
         if (needsVisibilityToggle && isHidden) {
diff --git a/packages/react-reconciler/src/ReactFiberWorkLoop.js b/packages/react-reconciler/src/ReactFiberWorkLoop.js
index ea8869a4a8c13..593546f2b262e 100644
--- a/packages/react-reconciler/src/ReactFiberWorkLoop.js
+++ b/packages/react-reconciler/src/ReactFiberWorkLoop.js
@@ -1506,11 +1506,8 @@ export function discreteUpdates<A, B, C, D, R>(
 
 // Overload the definition to the two valid signatures.
 // Warning, this opts-out of checking the function body.
-// eslint-disable-next-line no-unused-vars
 declare function flushSync<R>(fn: () => R): R;
-// eslint-disable-next-line no-redeclare
 declare function flushSync(void): void;
-// eslint-disable-next-line no-redeclare
 export function flushSync<R>(fn: (() => R) | void): R | void {
   // In legacy mode, we flush pending passive effects at the beginning of the
   // next event, not at the end of the previous one.
diff --git a/packages/react-reconciler/src/__tests__/ReactEffectOrdering-test.js b/packages/react-reconciler/src/__tests__/ReactEffectOrdering-test.js
index 259492f294cfc..a328f1309126b 100644
--- a/packages/react-reconciler/src/__tests__/ReactEffectOrdering-test.js
+++ b/packages/react-reconciler/src/__tests__/ReactEffectOrdering-test.js
@@ -8,8 +8,6 @@
  * @jest-environment node
  */
 
-/* eslint-disable no-func-assign */
-
 'use strict';
 
 let React;
diff --git a/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js b/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js
index b63a8b23476e4..0f4693789a6ef 100644
--- a/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js
+++ b/packages/react-reconciler/src/__tests__/ReactHooks-test.internal.js
@@ -1588,7 +1588,6 @@ describe('ReactHooks', () => {
 
       it(`warns on using differently ordered hooks (${hookNameA}, ${hookNameB}) on subsequent renders`, async () => {
         function App(props) {
-          /* eslint-disable no-unused-vars */
           if (props.update) {
             secondHelper();
             firstHelper();
@@ -1599,7 +1598,6 @@ describe('ReactHooks', () => {
           // This should not appear in the warning message because it occurs after the first mismatch
           useRefHelper();
           return null;
-          /* eslint-enable no-unused-vars */
         }
         let root;
         await act(() => {
@@ -1642,7 +1640,6 @@ describe('ReactHooks', () => {
 
       it(`warns when more hooks (${hookNameA}, ${hookNameB}) are used during update than mount`, async () => {
         function App(props) {
-          /* eslint-disable no-unused-vars */
           if (props.update) {
             firstHelper();
             secondHelper();
@@ -1650,7 +1647,6 @@ describe('ReactHooks', () => {
             firstHelper();
           }
           return null;
-          /* eslint-enable no-unused-vars */
         }
         let root;
         await act(() => {
@@ -1698,7 +1694,6 @@ describe('ReactHooks', () => {
 
       it(`warns when fewer hooks (${hookNameA}, ${hookNameB}) are used during update than mount`, async () => {
         function App(props) {
-          /* eslint-disable no-unused-vars */
           if (props.update) {
             firstHelper();
           } else {
@@ -1706,7 +1701,6 @@ describe('ReactHooks', () => {
             secondHelper();
           }
           return null;
-          /* eslint-enable no-unused-vars */
         }
         let root;
         await act(() => {
@@ -1728,7 +1722,6 @@ describe('ReactHooks', () => {
         '(useImperativeHandleHelper, useMemoHelper) on subsequent renders',
       async () => {
         function App(props) {
-          /* eslint-disable no-unused-vars */
           if (props.update) {
             useMemoHelper();
             useImperativeHandleHelper();
@@ -1739,7 +1732,6 @@ describe('ReactHooks', () => {
           // This should not appear in the warning message because it occurs after the first mismatch
           useRefHelper();
           return null;
-          /* eslint-enable no-unused-vars */
         }
         let root;
         await act(() => {
@@ -1781,7 +1773,6 @@ describe('ReactHooks', () => {
         useState(0);
       }
       function App(props) {
-        /* eslint-disable no-unused-vars */
         if (props.update) {
           useCustomHook();
           useReducer((s, a) => a, 0);
@@ -1791,7 +1782,6 @@ describe('ReactHooks', () => {
           useCustomHook();
         }
         return null;
-        /* eslint-enable no-unused-vars */
       }
       let root;
       await act(() => {
diff --git a/packages/react-reconciler/src/__tests__/useEffectEvent-test.js b/packages/react-reconciler/src/__tests__/useEffectEvent-test.js
index 38cdae73f0124..48c3dfae28f24 100644
--- a/packages/react-reconciler/src/__tests__/useEffectEvent-test.js
+++ b/packages/react-reconciler/src/__tests__/useEffectEvent-test.js
@@ -8,8 +8,6 @@
  * @jest-environment node
  */
 
-/* eslint-disable no-func-assign */
-
 'use strict';
 
 import {useInsertionEffect} from 'react';
diff --git a/packages/react-reconciler/src/__tests__/useRef-test.internal.js b/packages/react-reconciler/src/__tests__/useRef-test.internal.js
index 9c918b961ad31..748034399cc6a 100644
--- a/packages/react-reconciler/src/__tests__/useRef-test.internal.js
+++ b/packages/react-reconciler/src/__tests__/useRef-test.internal.js
@@ -8,8 +8,6 @@
  * @jest-environment node
  */
 
-/* eslint-disable no-func-assign */
-
 'use strict';
 
 describe('useRef', () => {
diff --git a/packages/react-reconciler/src/forks/ReactFiberConfig.custom.js b/packages/react-reconciler/src/forks/ReactFiberConfig.custom.js
index 492b1a4a8906f..973e460d03b30 100644
--- a/packages/react-reconciler/src/forks/ReactFiberConfig.custom.js
+++ b/packages/react-reconciler/src/forks/ReactFiberConfig.custom.js
@@ -24,21 +24,21 @@
 // really an argument to a top-level wrapping function.
 
 declare const $$$config: any;
-export opaque type Type = mixed; // eslint-disable-line no-undef
-export opaque type Props = mixed; // eslint-disable-line no-undef
-export opaque type Container = mixed; // eslint-disable-line no-undef
-export opaque type Instance = mixed; // eslint-disable-line no-undef
-export opaque type TextInstance = mixed; // eslint-disable-line no-undef
-export opaque type SuspenseInstance = mixed; // eslint-disable-line no-undef
-export opaque type HydratableInstance = mixed; // eslint-disable-line no-undef
-export opaque type PublicInstance = mixed; // eslint-disable-line no-undef
-export opaque type HostContext = mixed; // eslint-disable-line no-undef
-export opaque type UpdatePayload = mixed; // eslint-disable-line no-undef
-export opaque type ChildSet = mixed; // eslint-disable-line no-undef
-export opaque type TimeoutHandle = mixed; // eslint-disable-line no-undef
-export opaque type NoTimeout = mixed; // eslint-disable-line no-undef
-export opaque type RendererInspectionConfig = mixed; // eslint-disable-line no-undef
-export opaque type TransitionStatus = mixed; // eslint-disable-line no-undef
+export opaque type Type = mixed;
+export opaque type Props = mixed;
+export opaque type Container = mixed;
+export opaque type Instance = mixed;
+export opaque type TextInstance = mixed;
+export opaque type SuspenseInstance = mixed;
+export opaque type HydratableInstance = mixed;
+export opaque type PublicInstance = mixed;
+export opaque type HostContext = mixed;
+export opaque type UpdatePayload = mixed;
+export opaque type ChildSet = mixed;
+export opaque type TimeoutHandle = mixed;
+export opaque type NoTimeout = mixed;
+export opaque type RendererInspectionConfig = mixed;
+export opaque type TransitionStatus = mixed;
 export type EventResponder = any;
 
 export const getPublicInstance = $$$config.getPublicInstance;
@@ -179,7 +179,7 @@ export const validateHydratableTextInstance =
 //     (optional)
 // -------------------
 export type HoistableRoot = mixed;
-export type Resource = mixed; // eslint-disable-line no-undef
+export type Resource = mixed;
 export const supportsResources = $$$config.supportsResources;
 export const isHostHoistableType = $$$config.isHostHoistableType;
 export const getHoistableRoot = $$$config.getHoistableRoot;
diff --git a/packages/react-refresh/src/__tests__/ReactFresh-test.js b/packages/react-refresh/src/__tests__/ReactFresh-test.js
index 13ded584199e4..fa55e819aa5aa 100644
--- a/packages/react-refresh/src/__tests__/ReactFresh-test.js
+++ b/packages/react-refresh/src/__tests__/ReactFresh-test.js
@@ -7,8 +7,6 @@
  * @emails react-core
  */
 
-/* eslint-disable no-for-of-loops/no-for-of-loops */
-
 'use strict';
 
 let React;
diff --git a/packages/react-refresh/src/__tests__/ReactFreshIntegration-test.js b/packages/react-refresh/src/__tests__/ReactFreshIntegration-test.js
index c1e5f308b2202..57d9e5ca31d5b 100644
--- a/packages/react-refresh/src/__tests__/ReactFreshIntegration-test.js
+++ b/packages/react-refresh/src/__tests__/ReactFreshIntegration-test.js
@@ -7,8 +7,6 @@
  * @emails react-core
  */
 
-/* eslint-disable no-for-of-loops/no-for-of-loops */
-
 'use strict';
 
 let React;
diff --git a/packages/react-server-dom-webpack/src/ReactFlightDOMServerNode.js b/packages/react-server-dom-webpack/src/ReactFlightDOMServerNode.js
index 7546e1eac6612..a8e30d7b8c66a 100644
--- a/packages/react-server-dom-webpack/src/ReactFlightDOMServerNode.js
+++ b/packages/react-server-dom-webpack/src/ReactFlightDOMServerNode.js
@@ -55,7 +55,6 @@ function createDrainHandler(destination: Destination, request: Request) {
 function createCancelHandler(request: Request, reason: string) {
   return () => {
     stopFlowing(request);
-    // eslint-disable-next-line react-internal/prod-error-codes
     abort(request, new Error(reason));
   };
 }
diff --git a/packages/react-server/src/ReactFlightServerTemporaryReferences.js b/packages/react-server/src/ReactFlightServerTemporaryReferences.js
index c133f9030431d..643440cf02d37 100644
--- a/packages/react-server/src/ReactFlightServerTemporaryReferences.js
+++ b/packages/react-server/src/ReactFlightServerTemporaryReferences.js
@@ -83,7 +83,6 @@ export function createTemporaryReference<T>(id: string): TemporaryReference<T> {
   const reference: TemporaryReference<any> = Object.defineProperties(
     (function () {
       throw new Error(
-        // eslint-disable-next-line react-internal/safe-string-coercion
         `Attempted to call a temporary Client Reference from the server but it is on the client. ` +
           `It's not possible to invoke a client function from the server, it can ` +
           `only be rendered as a Component or passed to props of a Client Component.`,
diff --git a/packages/react-server/src/forks/ReactFizzConfig.custom.js b/packages/react-server/src/forks/ReactFizzConfig.custom.js
index 495a74fb18656..07ebb3295f32f 100644
--- a/packages/react-server/src/forks/ReactFizzConfig.custom.js
+++ b/packages/react-server/src/forks/ReactFizzConfig.custom.js
@@ -27,7 +27,7 @@ import type {Request} from 'react-server/src/ReactFizzServer';
 import type {TransitionStatus} from 'react-reconciler/src/ReactFiberConfig';
 
 declare const $$$config: any;
-export opaque type Destination = mixed; // eslint-disable-line no-undef
+export opaque type Destination = mixed;
 export opaque type RenderState = mixed;
 export opaque type HoistableState = mixed;
 export opaque type ResumableState = mixed;
diff --git a/packages/react-server/src/forks/ReactServerStreamConfig.custom.js b/packages/react-server/src/forks/ReactServerStreamConfig.custom.js
index 22cd6551c0b28..ce30b3fb771de 100644
--- a/packages/react-server/src/forks/ReactServerStreamConfig.custom.js
+++ b/packages/react-server/src/forks/ReactServerStreamConfig.custom.js
@@ -24,11 +24,11 @@
 // really an argument to a top-level wrapping function.
 
 declare const $$$config: any;
-export opaque type Destination = mixed; // eslint-disable-line no-undef
+export opaque type Destination = mixed;
 
-export opaque type PrecomputedChunk = mixed; // eslint-disable-line no-undef
-export opaque type Chunk = mixed; // eslint-disable-line no-undef
-export opaque type BinaryChunk = mixed; // eslint-disable-line no-undef
+export opaque type PrecomputedChunk = mixed;
+export opaque type Chunk = mixed;
+export opaque type BinaryChunk = mixed;
 
 export const scheduleWork = $$$config.scheduleWork;
 export const beginWriting = $$$config.beginWriting;
diff --git a/packages/react-test-renderer/src/ReactTestRenderer.js b/packages/react-test-renderer/src/ReactTestRenderer.js
index 7368eada24ad5..dd3b8627e1ee1 100644
--- a/packages/react-test-renderer/src/ReactTestRenderer.js
+++ b/packages/react-test-renderer/src/ReactTestRenderer.js
@@ -106,11 +106,10 @@ function toJSON(inst: Instance | TextInstance): ReactTestRendererNode | null {
     case 'TEXT':
       return inst.text;
     case 'INSTANCE': {
-      /* eslint-disable no-unused-vars */
       // We don't include the `children` prop in JSON.
       // Instead, we will include the actual rendered children.
+      // eslint-disable-next-line no-unused-vars
       const {children, ...props} = inst.props;
-      /* eslint-enable */
       let renderedChildren = null;
       if (inst.children && inst.children.length) {
         for (let i = 0; i < inst.children.length; i++) {
@@ -655,7 +654,6 @@ injectIntoDevTools({
 export {
   Scheduler as _Scheduler,
   create,
-  /* eslint-disable-next-line camelcase */
   batchedUpdates as unstable_batchedUpdates,
   act,
 };
diff --git a/packages/react/src/__tests__/ReactStrictMode-test.js b/packages/react/src/__tests__/ReactStrictMode-test.js
index 588b65f139ebe..724502c321674 100644
--- a/packages/react/src/__tests__/ReactStrictMode-test.js
+++ b/packages/react/src/__tests__/ReactStrictMode-test.js
@@ -625,7 +625,6 @@ describe('Concurrent Mode', () => {
       async () => await act(() => root.render(<StrictRoot />)),
     ).toErrorDev(
       [
-        /* eslint-disable max-len */
         `Warning: Using UNSAFE_componentWillMount in strict mode is not recommended and may indicate bugs in your code. See https://react.dev/link/unsafe-component-lifecycles for details.
 
 * Move code with side effects to componentDidMount, and set initial state in the constructor.
@@ -642,7 +641,6 @@ Please update the following components: Bar, Foo`,
 * Move data fetching code or side effects to componentDidUpdate.
 
 Please update the following components: App`,
-        /* eslint-enable max-len */
       ],
       {withoutStack: true},
     );
@@ -689,7 +687,6 @@ Please update the following components: App`,
         async () => await act(() => root.render(<StrictRoot />)),
       ).toErrorDev(
         [
-          /* eslint-disable max-len */
           `Warning: Using UNSAFE_componentWillMount in strict mode is not recommended and may indicate bugs in your code. See https://react.dev/link/unsafe-component-lifecycles for details.
 
 * Move code with side effects to componentDidMount, and set initial state in the constructor.
@@ -706,13 +703,11 @@ Please update the following components: Child`,
 * Move data fetching code or side effects to componentDidUpdate.
 
 Please update the following components: App`,
-          /* eslint-enable max-len */
         ],
         {withoutStack: true},
       );
     }).toWarnDev(
       [
-        /* eslint-disable max-len */
         `Warning: componentWillMount has been renamed, and is not recommended for use. See https://react.dev/link/unsafe-component-lifecycles for details.
 
 * Move code with side effects to componentDidMount, and set initial state in the constructor.
@@ -732,7 +727,6 @@ Please update the following components: Parent`,
 * Rename componentWillUpdate to UNSAFE_componentWillUpdate to suppress this warning in non-strict mode. In React 18.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run \`npx react-codemod rename-unsafe-lifecycles\` in your project source folder.
 
 Please update the following components: Parent`,
-        /* eslint-enable max-len */
       ],
       {withoutStack: true},
     );
diff --git a/packages/scheduler/npm/umd/scheduler.development.js b/packages/scheduler/npm/umd/scheduler.development.js
index f7d2e7cd4cf0b..873397d274118 100644
--- a/packages/scheduler/npm/umd/scheduler.development.js
+++ b/packages/scheduler/npm/umd/scheduler.development.js
@@ -7,8 +7,6 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-/* eslint-disable max-len */
-
 'use strict';
 
 (function (global, factory) {
diff --git a/packages/scheduler/npm/umd/scheduler.production.min.js b/packages/scheduler/npm/umd/scheduler.production.min.js
index 1a1f5f5ee0b50..56064ff7bb6f2 100644
--- a/packages/scheduler/npm/umd/scheduler.production.min.js
+++ b/packages/scheduler/npm/umd/scheduler.production.min.js
@@ -7,8 +7,6 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-/* eslint-disable max-len */
-
 'use strict';
 
 (function (global, factory) {
diff --git a/packages/scheduler/npm/umd/scheduler.profiling.min.js b/packages/scheduler/npm/umd/scheduler.profiling.min.js
index 1a1f5f5ee0b50..56064ff7bb6f2 100644
--- a/packages/scheduler/npm/umd/scheduler.profiling.min.js
+++ b/packages/scheduler/npm/umd/scheduler.profiling.min.js
@@ -7,8 +7,6 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-/* eslint-disable max-len */
-
 'use strict';
 
 (function (global, factory) {
diff --git a/packages/scheduler/src/__tests__/Scheduler-test.js b/packages/scheduler/src/__tests__/Scheduler-test.js
index d2695d7e04600..5af59a24a24f5 100644
--- a/packages/scheduler/src/__tests__/Scheduler-test.js
+++ b/packages/scheduler/src/__tests__/Scheduler-test.js
@@ -8,8 +8,6 @@
  * @jest-environment node
  */
 
-/* eslint-disable no-for-of-loops/no-for-of-loops */
-
 'use strict';
 
 let Scheduler;
diff --git a/packages/scheduler/src/__tests__/SchedulerSetImmediate-test.js b/packages/scheduler/src/__tests__/SchedulerSetImmediate-test.js
index bf521b15d3664..f32d6b3fe2b22 100644
--- a/packages/scheduler/src/__tests__/SchedulerSetImmediate-test.js
+++ b/packages/scheduler/src/__tests__/SchedulerSetImmediate-test.js
@@ -8,8 +8,6 @@
  * @jest-environment node
  */
 
-/* eslint-disable no-for-of-loops/no-for-of-loops */
-
 'use strict';
 
 let Scheduler;
diff --git a/packages/shared/ConsolePatchingDev.js b/packages/shared/ConsolePatchingDev.js
index 9af70b37adb2b..fe08bb7129fb1 100644
--- a/packages/shared/ConsolePatchingDev.js
+++ b/packages/shared/ConsolePatchingDev.js
@@ -52,7 +52,6 @@ export function disableLogs(): void {
         groupCollapsed: props,
         groupEnd: props,
       });
-      /* eslint-enable react-internal/no-production-logging */
     }
     disabledDepth++;
   }
@@ -62,7 +61,6 @@ export function reenableLogs(): void {
   if (__DEV__) {
     disabledDepth--;
     if (disabledDepth === 0) {
-      /* eslint-disable react-internal/no-production-logging */
       const props = {
         configurable: true,
         enumerable: true,
@@ -78,7 +76,6 @@ export function reenableLogs(): void {
         groupCollapsed: {...props, value: prevGroupCollapsed},
         groupEnd: {...props, value: prevGroupEnd},
       });
-      /* eslint-enable react-internal/no-production-logging */
     }
     if (disabledDepth < 0) {
       console.error(
diff --git a/packages/shared/isArray.js b/packages/shared/isArray.js
index f1f500c011290..e2d3f6b17eb32 100644
--- a/packages/shared/isArray.js
+++ b/packages/shared/isArray.js
@@ -11,7 +11,6 @@ declare function isArray(a: mixed): boolean %checks(Array.isArray(a));
 
 const isArrayImpl = Array.isArray;
 
-// eslint-disable-next-line no-redeclare
 function isArray(a: mixed): boolean {
   return isArrayImpl(a);
 }
diff --git a/packages/shared/reportGlobalError.js b/packages/shared/reportGlobalError.js
index 6cdb13d6a14ac..fd3d0b34ab6a7 100644
--- a/packages/shared/reportGlobalError.js
+++ b/packages/shared/reportGlobalError.js
@@ -45,7 +45,6 @@ const reportGlobalError: (error: mixed) => void =
           process.emit('uncaughtException', error);
           return;
         }
-        // eslint-disable-next-line react-internal/no-production-logging
         console['error'](error);
       };
 
diff --git a/scripts/babel/transform-react-version-pragma.js b/scripts/babel/transform-react-version-pragma.js
index 90c235ce480c9..76036f1075eba 100644
--- a/scripts/babel/transform-react-version-pragma.js
+++ b/scripts/babel/transform-react-version-pragma.js
@@ -1,7 +1,5 @@
 'use strict';
 
-/* eslint-disable no-for-of-loops/no-for-of-loops */
-
 const getComments = require('./getComments');
 
 const GATE_VERSION_STR = '@reactVersion ';
diff --git a/scripts/error-codes/__tests__/transform-error-messages.js b/scripts/error-codes/__tests__/transform-error-messages.js
index 1586e7dfbf8ed..c0355c1bb9063 100644
--- a/scripts/error-codes/__tests__/transform-error-messages.js
+++ b/scripts/error-codes/__tests__/transform-error-messages.js
@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  */
-/* eslint-disable quotes */
+
 'use strict';
 
 let babel = require('@babel/core');
diff --git a/scripts/flow/react-native-host-hooks.js b/scripts/flow/react-native-host-hooks.js
index 9c86a8ffeb95c..4012f78c4749e 100644
--- a/scripts/flow/react-native-host-hooks.js
+++ b/scripts/flow/react-native-host-hooks.js
@@ -7,8 +7,6 @@
  * @flow
  */
 
-/* eslint-disable */
-
 // libdefs cannot actually import. These are supposed to be the types imported
 // from 'react-native-renderer/src/ReactNativeTypes'
 type __MeasureOnSuccessCallback = any;
@@ -167,10 +165,12 @@ declare module 'react-native/Libraries/ReactPrivate/ReactNativePrivateInterface'
 declare module 'react-native/Libraries/ReactPrivate/ReactNativePrivateInitializeCore' {
 }
 
+// eslint-disable-next-line no-unused-vars
 declare const RN$enableMicrotasksInReact: boolean;
 
 // This is needed for a short term solution.
 // See https://github.com/facebook/react/pull/15490 for more info
+// eslint-disable-next-line no-unused-vars
 declare const nativeFabricUIManager: {
   createNode: (
     reactTag: number,
diff --git a/scripts/jest/devtools/setupEnv.js b/scripts/jest/devtools/setupEnv.js
index eadbbcb0afaed..7f838a9bc633a 100644
--- a/scripts/jest/devtools/setupEnv.js
+++ b/scripts/jest/devtools/setupEnv.js
@@ -47,7 +47,6 @@ global._test_react_version_focus = (range, testName, callback) => {
   const shouldPass = semver.satisfies(ReactVersionTestingAgainst, range);
 
   if (shouldPass) {
-    // eslint-disable-next-line jest/no-focused-tests
     test.only(testName, callback);
   } else {
     test.skip(testName, callback);