Skip to content

Adds type and lint checks to CI #5

Adds type and lint checks to CI

Adds type and lint checks to CI #5

Triggered via pull request September 27, 2024 13:08
Status Failure
Total duration 1m 2s
Artifacts

static_checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

20 errors and 10 warnings
typecheck: app.config.ts#L16
Cannot find module './.solid/tree' or its corresponding type declarations.
typecheck: app.config.ts#L17
Cannot find module './.solid/flat-entries' or its corresponding type declarations.
typecheck: app.config.ts#L18
Cannot find module './.solid/solid-start-flat-entries' or its corresponding type declarations.
typecheck: app.config.ts#L19
Cannot find module './.solid/solid-router-flat-entries' or its corresponding type declarations.
typecheck: app.config.ts#L20
Cannot find module './.solid/solid-meta-flat-entries' or its corresponding type declarations.
typecheck: app.config.ts#L21
Cannot find module './.solid/solid-router-tree' or its corresponding type declarations.
typecheck: app.config.ts#L22
Cannot find module './.solid/solid-start-tree' or its corresponding type declarations.
typecheck: app.config.ts#L23
Cannot find module './.solid/solid-meta-tree' or its corresponding type declarations.
typecheck: src/app.tsx#L23
Object is possibly 'undefined'.
typecheck: src/app.tsx#L24
Object is possibly 'undefined'.
lint: app.config.ts#L61
Unnecessary escape character: \g
lint: src/data/get-nav.tsx#L1
'matter' is defined but never used
lint: src/data/page-state.tsx#L1
'createEffect' is defined but never used
lint: src/data/page-state.tsx#L2
'StoreSetter' is defined but never used
lint: src/data/theme-provider.tsx#L40
Optional chain expressions can return undefined by design - using a non-null assertion is unsafe and wrong
lint: src/i18n/dictionaries/index.ts#L2
'ptbr' is defined but never used
lint: src/i18n/helpers.ts#L1
'useMatch' is defined but never used
lint: src/i18n/helpers.ts#L15
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
lint: src/i18n/translator.ts#L11
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
lint: src/ui/i18n-anchor.tsx#L18
'_' is assigned a value but never used
setup
Cache save failed.
lint: src/ui/eraser-link/index.tsx#L34
The reactive variable 'props.linkData.workspaceId' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L35
The reactive variable 'props.linkData.elementsId' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L36
The reactive variable 'props.linkData.elementsId' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L47
The reactive variable 'props.children' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L48
The reactive variable 'props.children' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L48
The reactive variable 'props.children[0]' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L54
Solid components run once, so an early return breaks reactivity. Move the condition inside a JSX element, such as a fragment or <Show />
lint: src/ui/eraser-link/index.tsx#L86
The reactive variable 'props.href' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
lint: src/ui/eraser-link/index.tsx#L88
Solid components run once, so an early return breaks reactivity. Move the condition inside a JSX element, such as a fragment or <Show />