diff options
author | Kristóf Marussy <kristof@marussy.com> | 2022-01-09 22:16:29 +0100 |
---|---|---|
committer | Kristóf Marussy <kristof@marussy.com> | 2022-01-09 22:17:26 +0100 |
commit | fb7118ff1c8f0dcd61f15e51b193512283d83fa1 (patch) | |
tree | 721cee6a64b44a56b7f05f39750a65cda5fb4ef6 /packages/renderer | |
parent | build: Add eslint-plugin-jest (diff) | |
download | sophie-fb7118ff1c8f0dcd61f15e51b193512283d83fa1.tar.gz sophie-fb7118ff1c8f0dcd61f15e51b193512283d83fa1.tar.zst sophie-fb7118ff1c8f0dcd61f15e51b193512283d83fa1.zip |
build: Add eslint-plugin-unicorn
Signed-off-by: Kristóf Marussy <kristof@marussy.com>
Diffstat (limited to 'packages/renderer')
-rw-r--r-- | packages/renderer/src/components/BrowserViewPlaceholder.tsx | 6 | ||||
-rw-r--r-- | packages/renderer/src/components/StoreProvider.tsx | 5 | ||||
-rw-r--r-- | packages/renderer/src/index.tsx | 4 | ||||
-rw-r--r-- | packages/renderer/src/stores/RendererStore.ts | 4 | ||||
-rw-r--r-- | packages/renderer/vite.config.js | 10 |
5 files changed, 15 insertions, 14 deletions
diff --git a/packages/renderer/src/components/BrowserViewPlaceholder.tsx b/packages/renderer/src/components/BrowserViewPlaceholder.tsx index 8f055e7..58407ee 100644 --- a/packages/renderer/src/components/BrowserViewPlaceholder.tsx +++ b/packages/renderer/src/components/BrowserViewPlaceholder.tsx | |||
@@ -44,15 +44,15 @@ export default observer(() => { | |||
44 | [store], | 44 | [store], |
45 | ); | 45 | ); |
46 | 46 | ||
47 | const resizeObserverRef = useRef<ResizeObserver | null>(null); | 47 | const resizeObserverRef = useRef<ResizeObserver | undefined>(); |
48 | 48 | ||
49 | const ref = useCallback( | 49 | const ref = useCallback( |
50 | (element: HTMLElement | null) => { | 50 | (element: HTMLElement | null) => { |
51 | if (resizeObserverRef.current !== null) { | 51 | if (resizeObserverRef.current !== undefined) { |
52 | resizeObserverRef.current.disconnect(); | 52 | resizeObserverRef.current.disconnect(); |
53 | } | 53 | } |
54 | if (element === null) { | 54 | if (element === null) { |
55 | resizeObserverRef.current = null; | 55 | resizeObserverRef.current = undefined; |
56 | return; | 56 | return; |
57 | } | 57 | } |
58 | resizeObserverRef.current = new ResizeObserver(onResize); | 58 | resizeObserverRef.current = new ResizeObserver(onResize); |
diff --git a/packages/renderer/src/components/StoreProvider.tsx b/packages/renderer/src/components/StoreProvider.tsx index bb8495c..3360a43 100644 --- a/packages/renderer/src/components/StoreProvider.tsx +++ b/packages/renderer/src/components/StoreProvider.tsx | |||
@@ -22,11 +22,12 @@ import React, { createContext, useContext } from 'react'; | |||
22 | 22 | ||
23 | import type { RendererStore } from '../stores/RendererStore'; | 23 | import type { RendererStore } from '../stores/RendererStore'; |
24 | 24 | ||
25 | const StoreContext = createContext<RendererStore | null>(null); | 25 | // eslint-disable-next-line unicorn/no-useless-undefined -- `createContext` expects 1 parameter. |
26 | const StoreContext = createContext<RendererStore | undefined>(undefined); | ||
26 | 27 | ||
27 | export function useStore(): RendererStore { | 28 | export function useStore(): RendererStore { |
28 | const store = useContext(StoreContext); | 29 | const store = useContext(StoreContext); |
29 | if (store === null) { | 30 | if (store === undefined) { |
30 | throw new Error('useStore can only be called inside of StoreProvider'); | 31 | throw new Error('useStore can only be called inside of StoreProvider'); |
31 | } | 32 | } |
32 | return store; | 33 | return store; |
diff --git a/packages/renderer/src/index.tsx b/packages/renderer/src/index.tsx index d900e50..a42a30c 100644 --- a/packages/renderer/src/index.tsx +++ b/packages/renderer/src/index.tsx | |||
@@ -45,8 +45,8 @@ if (isDevelopment) { | |||
45 | const store = createAndConnectRendererStore(window.sophieRenderer); | 45 | const store = createAndConnectRendererStore(window.sophieRenderer); |
46 | 46 | ||
47 | if (isDevelopment) { | 47 | if (isDevelopment) { |
48 | exposeToReduxDevtools(store).catch((err) => { | 48 | exposeToReduxDevtools(store).catch((error) => { |
49 | log.error('Cannot initialize redux devtools', err); | 49 | log.error('Cannot initialize redux devtools', error); |
50 | }); | 50 | }); |
51 | } | 51 | } |
52 | 52 | ||
diff --git a/packages/renderer/src/stores/RendererStore.ts b/packages/renderer/src/stores/RendererStore.ts index 0b78ce1..4cc5163 100644 --- a/packages/renderer/src/stores/RendererStore.ts +++ b/packages/renderer/src/stores/RendererStore.ts | |||
@@ -86,8 +86,8 @@ export function createAndConnectRendererStore( | |||
86 | applyPatch(store.shared, patch); | 86 | applyPatch(store.shared, patch); |
87 | }, | 87 | }, |
88 | }) | 88 | }) |
89 | .catch((err) => { | 89 | .catch((error) => { |
90 | log.error('Failed to connect to shared store', err); | 90 | log.error('Failed to connect to shared store', error); |
91 | }); | 91 | }); |
92 | 92 | ||
93 | return store; | 93 | return store; |
diff --git a/packages/renderer/vite.config.js b/packages/renderer/vite.config.js index e20e0f1..cb0203c 100644 --- a/packages/renderer/vite.config.js +++ b/packages/renderer/vite.config.js | |||
@@ -1,15 +1,15 @@ | |||
1 | /* eslint-disable no-process-env */ | 1 | /* eslint-disable no-process-env */ |
2 | /* eslint-env node */ | 2 | /* eslint-env node */ |
3 | 3 | ||
4 | import { builtinModules } from 'module'; | 4 | import { builtinModules } from 'node:module'; |
5 | import { join } from 'path'; | 5 | import path from 'node:path'; |
6 | 6 | ||
7 | import react from '@vitejs/plugin-react'; | 7 | import react from '@vitejs/plugin-react'; |
8 | 8 | ||
9 | import { banner, chrome } from '../../config/buildConstants.js'; | 9 | import { banner, chrome } from '../../config/buildConstants.js'; |
10 | import fileURLToDirname from '../../config/fileURLToDirname.js'; | 10 | import fileUrlToDirname from '../../config/fileUrlToDirname.js'; |
11 | 11 | ||
12 | const thisDir = fileURLToDirname(import.meta.url); | 12 | const thisDir = fileUrlToDirname(import.meta.url); |
13 | 13 | ||
14 | const mode = process.env.MODE || 'development'; | 14 | const mode = process.env.MODE || 'development'; |
15 | 15 | ||
@@ -24,7 +24,7 @@ export default { | |||
24 | logLevel: 'info', | 24 | logLevel: 'info', |
25 | mode, | 25 | mode, |
26 | root: thisDir, | 26 | root: thisDir, |
27 | cacheDir: join(thisDir, '../../.vite'), | 27 | cacheDir: path.join(thisDir, '../../.vite'), |
28 | plugins: [ | 28 | plugins: [ |
29 | react({ | 29 | react({ |
30 | babel: { | 30 | babel: { |