Skip to content

Story authoring tool #895

Story authoring tool

Story authoring tool #895

Triggered via pull request October 12, 2023 08:21
Status Failure
Total duration 2m 2s
Artifacts

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

9 errors and 14 warnings
ts-check: app/scripts/components/publication-tool/editor.tsx#L56
Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
ts-check: app/scripts/components/sandbox/mdx-editor/mdx-editor.tsx#L211
Object is possibly 'undefined'.
ts-check: app/scripts/components/sandbox/mdx-editor/mdx-editor.tsx#L311
Argument of type 'string[]' is not assignable to parameter of type 'string'.
ts-check
Process completed with exit code 2.
lint: app/scripts/components/sandbox/mdx-editor/index.tsx#L90
'savedSource' is assigned a value but never used
lint: app/scripts/components/sandbox/mdx-editor/mdx-editor.tsx#L49
'ResetButton' is assigned a value but never used
lint: app/scripts/components/sandbox/mdx-editor/mdx-editor.tsx#L352
Unexpected console statement
lint
Process completed with exit code 1.
prep
The following actions uses node12 which is deprecated and will be forced to run on node16: styfle/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
ts-check
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint: app/scripts/components/analysis/define/aoi-selector.tsx#L99
React Hook useCallback has a missing dependency: 'mapRef'. Either include it or remove the dependency array
lint: app/scripts/components/analysis/define/aoi-selector.tsx#L118
React Hook useEffect has a missing dependency: 'mapRef'. Either include it or remove the dependency array
lint: app/scripts/components/analysis/define/index.tsx#L105
Use a ! assertion to more succinctly remove null and undefined from the type
lint: app/scripts/components/analysis/define/index.tsx#L115
Use a ! assertion to more succinctly remove null and undefined from the type
lint: app/scripts/components/analysis/define/index.tsx#L170
Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator
lint: app/scripts/components/analysis/define/index.tsx#L200
Unnecessary optional chain on a non-nullish value
lint: app/scripts/components/analysis/define/index.tsx#L207
React Hook useEffect has a missing dependency: 'datasetsLayers'. Either include it or remove the dependency array
lint: app/scripts/components/analysis/define/use-custom-aoi.ts#L129
Unnecessary conditional, value is always falsy
lint: app/scripts/components/analysis/page-hero-media.tsx#L143
React Hook useEffect has a missing dependency: 'theme.color'. Either include it or remove the dependency array
lint: app/scripts/components/analysis/page-hero-media.tsx#L159
Unexpected any. Specify a different type