From 4da86eb643c576fd4e7aa26c6bc89fb3777bcd60 Mon Sep 17 00:00:00 2001
From: Joost De Cock <joost@joost.at>
Date: Sun, 13 Feb 2022 16:10:51 +0100
Subject: [PATCH] chore: fixed linter warnings

---
 packages/freesewing.lab/hooks/useApp.js                     | 4 ----
 .../components/workbench/draft/part/index.js                | 2 +-
 .../workbench/menu/test-design-options/option-sub-group.js  | 4 ++--
 .../components/workbench/menu/test-design-options/option.js | 2 +-
 packages/freesewing.shared/components/wrappers/workbench.js | 2 +-
 packages/i18n/src/prebuild.mjs                              | 6 +++---
 6 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/packages/freesewing.lab/hooks/useApp.js b/packages/freesewing.lab/hooks/useApp.js
index 37d31ebaaa8..5e5e219cd6b 100644
--- a/packages/freesewing.lab/hooks/useApp.js
+++ b/packages/freesewing.lab/hooks/useApp.js
@@ -1,11 +1,9 @@
 import { useState } from 'react'
-import set from 'lodash.set'
 // Stores state in local storage
 import useLocalStorage from 'shared/hooks/useLocalStorage.js'
 // Patterns
 import patterns from 'site/patterns.json'
 // Locale and translation
-import { useRouter } from 'next/router'
 import { useTranslation } from 'next-i18next'
 import { capitalize } from 'shared/utils'
 
@@ -54,8 +52,6 @@ const initialNavigation = (t) => {
 function useApp(full = true) {
 
   // Locale (aka language)
-  const router = useRouter()
-  const { locale } = router
   const { t } = useTranslation(['app'])
 
   // User color scheme preference
diff --git a/packages/freesewing.shared/components/workbench/draft/part/index.js b/packages/freesewing.shared/components/workbench/draft/part/index.js
index ef2cce7c626..7fe4293367e 100644
--- a/packages/freesewing.shared/components/workbench/draft/part/index.js
+++ b/packages/freesewing.shared/components/workbench/draft/part/index.js
@@ -7,7 +7,7 @@ const XrayPart = props => {
   // Don't bother if this is the only part on display
   if (props.gist.only && props.gist.only.length === 1) return null
   const i = props.gist.xray?.reveal
-    ? Object.keys(props.gist.xray?.reveal).indexOf(props.partName)%10
+    ? Object.keys(props.gist.xray.reveal).indexOf(props.partName)%10
     : 0
   const { topLeft, bottomRight } = props.part
 
diff --git a/packages/freesewing.shared/components/workbench/menu/test-design-options/option-sub-group.js b/packages/freesewing.shared/components/workbench/menu/test-design-options/option-sub-group.js
index 05b28c595b2..aa1b0b11922 100644
--- a/packages/freesewing.shared/components/workbench/menu/test-design-options/option-sub-group.js
+++ b/packages/freesewing.shared/components/workbench/menu/test-design-options/option-sub-group.js
@@ -1,6 +1,6 @@
-import { linkClasses, Chevron } from 'shared/components/navigation/primary.js'
+import { Chevron } from 'shared/components/navigation/primary.js'
 import Option from './option'
-import { Li, Ul, Details, Summary, SumButton, SumDiv, Deg } from 'shared/components/workbench/menu'
+import { Li, Ul, Details, Summary, SumDiv, Deg } from 'shared/components/workbench/menu'
 import { useTranslation } from 'next-i18next'
 
 const OptionSubGroup = props => {
diff --git a/packages/freesewing.shared/components/workbench/menu/test-design-options/option.js b/packages/freesewing.shared/components/workbench/menu/test-design-options/option.js
index 1403746ba05..eadf6918ef3 100644
--- a/packages/freesewing.shared/components/workbench/menu/test-design-options/option.js
+++ b/packages/freesewing.shared/components/workbench/menu/test-design-options/option.js
@@ -1,5 +1,5 @@
 import { linkClasses } from 'shared/components/navigation/primary.js'
-import { Li, Ul, Details, Summary, Deg } from 'shared/components/workbench/menu'
+import { Li } from 'shared/components/workbench/menu'
 import { useTranslation } from 'next-i18next'
 
 const SumButton = props => (
diff --git a/packages/freesewing.shared/components/wrappers/workbench.js b/packages/freesewing.shared/components/wrappers/workbench.js
index 167d02d9a11..2ced6d41918 100644
--- a/packages/freesewing.shared/components/wrappers/workbench.js
+++ b/packages/freesewing.shared/components/wrappers/workbench.js
@@ -82,7 +82,7 @@ const WorkbenchWrapper = ({ app, pattern }) => {
   let draft = false
   if (['draft', 'events', 'test'].indexOf(gist?._state?.view) !== -1) {
     draft = new pattern(gist)
-    if (gist?.renderer === 'svg') draft.use(theme)
+    if (gist.renderer === 'svg') draft.use(theme)
     try {
       if (gist._state.view !== 'test') draft.draft()
     }
diff --git a/packages/i18n/src/prebuild.mjs b/packages/i18n/src/prebuild.mjs
index 008b7d500bb..c01e4ab44f4 100644
--- a/packages/i18n/src/prebuild.mjs
+++ b/packages/i18n/src/prebuild.mjs
@@ -1,7 +1,7 @@
 import yaml from 'js-yaml'
 import path from 'path'
 import rdir from 'recursive-readdir'
-import { readdir, readFile, writeFile } from 'fs/promises'
+import { readFile, writeFile } from 'fs/promises'
 import { fileURLToPath } from 'url'
 import { dirname } from 'path'
 
@@ -131,7 +131,7 @@ const ${name} = ${JSON.stringify(data, null ,2)}
 
 export default ${name}
 `
-const localeFile = (locale, namespaces) => `${header}
+const localeFile = (namespaces) => `${header}
 ${namespaces
   .map(ns => 'import '+ns+' from "./'+ns+'.mjs"')
   .join("\n")
@@ -180,7 +180,7 @@ const writeFiles = async allNamespaces => {
     promises.push(
       writeFile(
         path.resolve(__dirname, 'next', locale, 'index.mjs', ),
-        localeFile(locale, Object.keys(namespaces))
+        localeFile(Object.keys(namespaces))
       )
     )
   }