Skip to content

(spans) Removes redundant vitest imports #118

(spans) Removes redundant vitest imports

(spans) Removes redundant vitest imports #118

Triggered via pull request November 20, 2023 16:36
Status Success
Total duration 56s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

wc-tests.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
test: web-client/src/components/form-field.tsx#L28
The reactive variable 'props.onInput' should be wrapped in a function for reactivity. This includes event handler bindings on native elements, which are not reactive like other JSX props
test: web-client/src/components/sources/source-pane.tsx#L26
Forbidden non-null assertion
test: web-client/src/components/sources/source-pane.tsx#L33
Forbidden non-null assertion
test: web-client/src/components/span/span-detail.tsx#L40
Forbidden non-null assertion
test: web-client/src/components/span/span-detail.tsx#L50
Forbidden non-null assertion
test: web-client/src/components/span/span-detail.tsx#L56
Forbidden non-null assertion
test: web-client/src/components/span/span-detail.tsx#L58
Forbidden non-null assertion
test: web-client/src/components/status-indicator.tsx#L7
The reactive variable 'props.status' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored
test: web-client/src/components/status-indicator.tsx#L11
Empty components are self-closing
test: web-client/src/lib/span/find-spans-by-name.ts#L17
Forbidden non-null assertion