Skip to content

[github] create lint and type check workflow #1

[github] create lint and type check workflow

[github] create lint and type check workflow #1

Triggered via pull request August 10, 2023 19:04
Status Failure
Total duration 55s
Artifacts

lint-and-tsc.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 10 warnings
lint: apps/menu-bar/src/components/CircularProgress.tsx#L2
'Animated' is defined but never used
lint: apps/menu-bar/src/components/Text.tsx#L2
'darkTheme' is defined but never used
lint: apps/menu-bar/src/components/Text.tsx#L3
'lightTheme' is defined but never used
lint: apps/menu-bar/src/hooks/useDeviceAudioPreferences.ts#L23
React Hook useEffect has a missing dependency: 'getAudioPreferences'. Either include it or remove the dependency array
lint: apps/menu-bar/src/popover/Core.tsx#L79
'simulator' is defined but never used. Allowed unused args must match /^_/u
lint: apps/menu-bar/src/utils/theme.ts#L193
Replace `"800"` with `'800'`
lint: apps/menu-bar/src/utils/theme.ts#L205
Replace `"800"` with `'800'`
lint: apps/menu-bar/src/windows/Onboarding.tsx#L38
eval can be harmful
lint: apps/menu-bar/src/windows/Settings.tsx#L125
Replace `Copyright·650·Industries·Inc,·2023` with `⏎········Copyright·650·Industries·Inc,·2023⏎······`
lint
Process completed with exit code 1.
lint: apps/menu-bar/src/components/NativeColorPalette.tsx#L19
Inline style: { flexDirection: 'row', gap: 5, alignItems: 'center', padding: 5 }
lint: apps/menu-bar/src/components/NativeColorPalette.tsx#L52
Expected '!==' and instead saw '!='
lint: apps/menu-bar/src/components/PathInput.tsx#L31
Inline style: { opacity: 0.66 }
lint: apps/menu-bar/src/hooks/useDeviceAudioPreferences.ts#L10
'emulatorWithoutAudio' is already declared in the upper scope on line 7 column 10
lint: apps/menu-bar/src/hooks/useListDevices.ts#L17
Value of 'error' may be overwritten in IE 8 and earlier
lint: apps/menu-bar/src/hooks/useListDevices.ts#L17
'error' is already declared in the upper scope on line 10 column 10
lint: apps/menu-bar/src/popover/Core.tsx#L257
Do not define components during render. React will see a new component type on every render and destroy the entire subtree’s DOM nodes and state (https://reactjs.org/docs/reconciliation.html#elements-of-different-types). Instead, move this component definition out of the parent component “Core” and pass data as props. If you want to allow component creation in props, set allowAsProps option to true
lint: apps/menu-bar/src/utils/theme.ts#L193
Strings must use singlequote
lint: apps/menu-bar/src/utils/theme.ts#L205
Strings must use singlequote
lint: apps/menu-bar/src/windows/index.ts#L14
Unexpected use of '|'