chore(linter): disable no-console rule

MiracleHorizon · Aug 9, 2024 · a7315de · a7315de
1 parent 82cecf3
commit a7315de
Show file tree

Hide file tree

Showing 9 changed files with 8 additions and 203 deletions.
diff --git a/apps/backend/eslint.config.mjs b/apps/backend/eslint.config.mjs
@@ -1,16 +1,3 @@
 import defaultConfig from '../../eslint.config.js'
 
-/**
- TODO: Await issues updates:
- - react-hooks: https://github.com/facebook/react/issues/28313,
- - next: https://github.com/vercel/next.js/issues/64409
- */
-
-export default [
-  ...defaultConfig,
-  {
-    rules: {
-      'no-console': 'off'
-    }
-  }
-]
+export default defaultConfig
diff --git a/apps/backend/package.json b/apps/backend/package.json
@@ -46,13 +46,16 @@
     "@types/multer": "1.4.11",
     "@types/node": "20.12.2",
     "@types/supertest": "6.0.2",
-    "eslint": "8.57.0",
+    "eslint": "9.8.0",
     "jest": "29.7.0",
     "source-map-support": "0.5.21",
     "supertest": "6.3.4",
     "ts-jest": "29.1.2",
     "ts-loader": "9.5.1",
     "ts-node": "10.9.2",
     "typescript": "5.4.5"
+  },
+  "overrides": {
+    "eslint": "9.8.0"
   }
 }
diff --git a/apps/frontend/src/app/global-error.tsx b/apps/frontend/src/app/global-error.tsx
@@ -17,7 +17,6 @@ export const metadata: Metadata = {
 const GlobalError = ({ error, reset }: ErrorPageProps) => {
   const themeColor = getThemeColorClientCookie()
 
-  // eslint-disable-next-line no-console
   console.log(error)
 
   return (

diff --git a/apps/frontend/src/components/ImageUploadForm.tsx b/apps/frontend/src/components/ImageUploadForm.tsx
@@ -88,7 +88,6 @@ export const ImageUploadForm = ({
 
       onUpload(file)
     } catch (err) {
-      // eslint-disable-next-line no-console
       console.log(err)
       setError(new Error(DEFAULT_MESSAGE))
     }

diff --git a/apps/frontend/src/lib/ui/EyeDropper.tsx b/apps/frontend/src/lib/ui/EyeDropper.tsx
@@ -36,7 +36,6 @@ const EyeDropperComponent = ({
 
       setColor(color)
     } catch (err) {
-      // eslint-disable-next-line no-console
       console.warn(err)
     } finally {
       document.body.classList.remove('eye-dropper-active')

diff --git a/apps/frontend/src/stores/hooks/useImportSettings.ts b/apps/frontend/src/stores/hooks/useImportSettings.ts
@@ -23,7 +23,6 @@ export const useImportSettings = (selectedTab: ToolbarTab) => {
     try {
       return JSON.parse(settingsJSON)
     } catch (err) {
-      // eslint-disable-next-line no-console
       console.warn('Failed to parse settings JSON', err)
 
       return null
@@ -72,7 +71,6 @@ export const useImportSettings = (selectedTab: ToolbarTab) => {
       setData(settings)
       handleOpenConfirmAlert()
     } catch (err) {
-      // eslint-disable-next-line no-console
       console.warn('Failed to import settings', err)
 
       handleOpenValidationAlert()

diff --git a/apps/frontend/src/widgets/SettingsPanel/TabMetadata/TabMetadataContent.tsx b/apps/frontend/src/widgets/SettingsPanel/TabMetadata/TabMetadataContent.tsx
@@ -41,7 +41,6 @@ export const TabMetadataContent = () => {
         })
         .then(result => setParsedMetadata(result ?? null))
         .catch(err => {
-          // eslint-disable-next-line no-console
           console.log(err)
 
           setParsedMetadata(null)

diff --git a/eslint.config.js b/eslint.config.js
@@ -41,7 +41,7 @@ export default tsEslint.config(
       quotes: ['error', 'single'],
       semi: ['error', 'never'],
       eqeqeq: 'error',
-      'no-console': 'warn',
+      'no-console': 'off',
       'no-debugger': 'error',
       'no-unused-vars': 'error',
       'prefer-const': 'error',