Skip to content

Handle analysis/define datasets w/o collection summaries #1129

Handle analysis/define datasets w/o collection summaries

Handle analysis/define datasets w/o collection summaries #1129

Triggered via pull request January 10, 2024 16:49
Status Success
Total duration 1m 17s
Artifacts

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

14 warnings
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#L98
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#L117
React Hook useEffect has a missing dependency: 'mapRef'. Either include it or remove the dependency array
lint: app/scripts/components/analysis/define/index.tsx#L295
Prefer using nullish coalescing operator (`??`) instead of a logical or (`||`), as it is a safer operator
lint: app/scripts/components/analysis/define/index.tsx#L336
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-metrics-dropdown.tsx#L76
Unexpected any. Specify a different type
lint: app/scripts/components/analysis/results/concurrency.ts#L8
The use of method `shift` is not allowed as it might be a mutating method
lint: app/scripts/components/analysis/results/concurrency.ts#L28
The use of method `push` is not allowed as it might be a mutating method