Skip to content

Change Discovery related strings to Data Stories #557

Change Discovery related strings to Data Stories

Change Discovery related strings to Data Stories #557

Triggered via pull request July 5, 2023 15:04
Status Success
Total duration 1m 55s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
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#L104
Use a ! assertion to more succinctly remove null and undefined from the type
lint: app/scripts/components/analysis/define/index.tsx#L161
Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator
lint: app/scripts/components/analysis/define/index.tsx#L187
Unnecessary optional chain on a non-nullish value
lint: app/scripts/components/analysis/define/index.tsx#L194
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
lint: app/scripts/components/analysis/results/analysis-head-actions.tsx#L85
Unexpected any. Specify a different type