chore: remove unused ESLint directives (nodejs#2772)

forgeRW · Sep 19, 2022 · 9e0300d · 9e0300d
1 parent f32650f
commit 9e0300d
Show file tree

Hide file tree

Showing 12 changed files with 1 addition and 15 deletions.
diff --git a/gatsby-browser.tsx b/gatsby-browser.tsx
@@ -5,12 +5,10 @@ import type { WrapPageElementBrowser } from './src/types';
 
 import './src/styles/index.scss';
 
-// eslint-disable-next-line import/prefer-default-export
 export const wrapPageElement: WrapPageElementBrowser = ({ element, props }) => {
   const { locale = defaultLanguage, intlMessages = defaultMessages } =
     props.pageContext;
 
-  // eslint-disable-next-line react/jsx-props-no-spreading
   return (
     <ReactIntlProvider locale={locale} messages={intlMessages}>
       {element}

diff --git a/gatsby-ssr.tsx b/gatsby-ssr.tsx
@@ -3,12 +3,10 @@ import ReactIntlProvider from './src/containers/ReactIntl';
 import { defaultLanguage, defaultMessages } from './locales';
 import type { WrapPageElementSSR } from './src/types';
 
-// eslint-disable-next-line import/prefer-default-export
 export const wrapPageElement: WrapPageElementSSR = ({ element, props }) => {
   const { locale = defaultLanguage, intlMessages = defaultMessages } =
     props.pageContext;
 
-  // eslint-disable-next-line react/jsx-props-no-spreading
   return (
     <ReactIntlProvider locale={locale} messages={intlMessages}>
       {element}

diff --git a/package.json b/package.json
@@ -81,7 +81,7 @@
     "test-watch": "jest --watch",
     "test-ci": "npm run test -- --coverage",
     "lint": "npm run type-check && npm run lint:js && npm run lint:md && npm run lint:scss",
-    "lint:js": "eslint \"**/*.{js,jsx,ts,tsx}\" --cache --cache-file .eslintjscache",
+    "lint:js": "eslint \"**/*.{js,jsx,ts,tsx}\" --cache --cache-file .eslintjscache --report-unused-disable-directives",
     "lint:md": "eslint \"content/**/*.{md,mdx}\" --cache --cache-file .eslintmdcache",
     "lint:scss": "stylelint --config ./.stylelintrc.json \"./src/**/*.{css,sass,scss}\"",
     "lint:fix": "npm run lint -- --fix",

diff --git a/src/components/Seo/index.tsx b/src/components/Seo/index.tsx
@@ -1,6 +1,5 @@
 import React from 'react';
 import { Helmet } from 'react-helmet';
-// eslint-disable-next-line import/no-extraneous-dependencies
 import { useLocation } from '@gatsbyjs/reach-router';
 import config from '../../config.json';
 

diff --git a/src/hooks/useAutoClosableDropdown.tsx b/src/hooks/useAutoClosableDropdown.tsx
@@ -1,7 +1,6 @@
 import React, { useEffect, useRef } from 'react';
 import Dropdown, { DropdownItem } from '../components/Dropdown';
 
-// eslint-disable-next-line import/prefer-default-export
 export const useAutoClosableDropdown = <T extends HTMLElement>(
   items: Array<DropdownItem>,
   elementRef: React.RefObject<T>

diff --git a/src/hooks/useMediaQuery.tsx b/src/hooks/useMediaQuery.tsx
@@ -19,7 +19,6 @@ const mediaQueryChangeUnsubscribe = (
   }
 };
 
-// eslint-disable-next-line import/prefer-default-export
 export function useMediaQuery(query: string): boolean | undefined {
   const [matches, setMatches] = useState<boolean>();
 

diff --git a/src/hooks/useNavigateToDifferentLocale.ts b/src/hooks/useNavigateToDifferentLocale.ts
@@ -1,5 +1,4 @@
 import { navigate as gatsbyNavigate, withPrefix } from 'gatsby';
-// eslint-disable-next-line import/no-extraneous-dependencies
 import { useLocation } from '@gatsbyjs/reach-router';
 import { useLocalization } from 'gatsby-theme-i18n';
 

diff --git a/src/hooks/useNodeJsContributorsApi.tsx b/src/hooks/useNodeJsContributorsApi.tsx
@@ -133,7 +133,6 @@ export interface ContributorApiResponse {
   url: string;
   type: string;
   contributions: number;
-  /* eslint-disable camelcase */
   node_id: string;
   avatar_url: string;
   gravatar_id: string;

diff --git a/src/hooks/useOnScreen.tsx b/src/hooks/useOnScreen.tsx
@@ -6,7 +6,6 @@ import { MutableRefObject, useEffect, useState } from 'react';
  * @param ref
  * @param observeOnce
  */
-// eslint-disable-next-line import/prefer-default-export
 export function useOnScreen(
   ref: MutableRefObject<Element>,
   observeOnce: boolean

diff --git a/test/__mocks__/@testing-library/react.tsx b/test/__mocks__/@testing-library/react.tsx
@@ -13,9 +13,7 @@ const render = (
   ui: React.ReactElement,
   { locale = 'en', ...renderOptions } = {}
 ) => {
-  // eslint-disable-next-line react/prop-types
   const ProviderComponent = ({ children }: { children: React.ReactNode }) => (
-    // eslint-disable-next-line react/jsx-filename-extension
     <IntlProvider locale={locale} messages={messages}>
       {children}
     </IntlProvider>

diff --git a/util-node/apiDocsTransformUtils.js b/util-node/apiDocsTransformUtils.js
@@ -90,7 +90,6 @@ function createNavigationCreator({ version, name }) {
             navigationEntriesForFile.push(
               ...metadataTypes.map(type => ({
                 slug: `${apiPath}${version}/${name}/${getHash(type, title)}`,
-                // eslint-disable-next-line no-useless-escape
                 title: title.split('(')[0].replace(/[^\w\- ]+/g, ''),
                 type,
                 name,

diff --git a/util-node/createSlug.js b/util-node/createSlug.js
@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/explicit-function-return-type */
 function createSlug(title) {
   let slug = title.toLowerCase().trim();