Skip to content

Refactoring and improvements to work with encode images #129

Refactoring and improvements to work with encode images

Refactoring and improvements to work with encode images #129

Triggered via pull request July 17, 2023 20:14
Status Success
Total duration 3m 3s
Artifacts

checks.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

23 warnings
prep
The following actions uses node12 which is deprecated and will be forced to run on node16: styfle/[email protected], actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. 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, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test: src/components/EncodeCanvas.js#L144
React Hook useEffect has missing dependencies: 'activeEncodeImageItem', 'map', 'requestSAM', and 'showNotification'. Either include them or remove the dependency array
test: src/components/EncodeItem.js#L53
Redundant alt attribute. Screen-readers already announce `img` tags as an image. You don’t need to use the words `image`, `photo,` or `picture` (or any specified custom words) in the alt prop
test: src/components/EncodeItems.js#L69
React Hook useEffect has a missing dependency: 'dispatchEncodeItems'. Either include it or remove the dependency array
test: src/components/Header.js#L1
'useState' is defined but never used
test: src/components/Items.js#L45
React Hook useLayoutEffect has a missing dependency: 'dispatchSetItems'. Either include it or remove the dependency array
test: src/components/Projects.js#L43
React Hook useCallback has a missing dependency: 'dispatchActiveEncodeImageItem'. Either include it or remove the dependency array
test: src/components/Projects.js#L66
React Hook useEffect has missing dependencies: 'projects.features' and 'setProject'. Either include them or remove the dependency array
test: src/components/map/MagicWand.js#L58
React Hook useEffect has missing dependencies: 'activeClass.color', 'activeClass.name', 'activeProject.properties.name', 'dispatchSetItems', 'items', 'map', and 'wand'. Either include them or remove the dependency array
test: src/components/map/ProjectLayer.js#L109
React Hook useEffect has missing dependencies: 'dispatchSetPointsSelector' and 'map'. Either include them or remove the dependency array
test: src/components/map/ProjectLayer.js#L119
React Hook useEffect has a missing dependency: 'map'. Either include it or remove the dependency array
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@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: src/components/EncodeCanvas.js#L144
React Hook useEffect has missing dependencies: 'activeEncodeImageItem', 'map', 'requestSAM', and 'showNotification'. Either include them or remove the dependency array
lint: src/components/EncodeItem.js#L53
Redundant alt attribute. Screen-readers already announce `img` tags as an image. You don’t need to use the words `image`, `photo,` or `picture` (or any specified custom words) in the alt prop
lint: src/components/EncodeItems.js#L69
React Hook useEffect has a missing dependency: 'dispatchEncodeItems'. Either include it or remove the dependency array
lint: src/components/Header.js#L1
'useState' is defined but never used
lint: src/components/Items.js#L45
React Hook useLayoutEffect has a missing dependency: 'dispatchSetItems'. Either include it or remove the dependency array
lint: src/components/Projects.js#L43
React Hook useCallback has a missing dependency: 'dispatchActiveEncodeImageItem'. Either include it or remove the dependency array
lint: src/components/Projects.js#L66
React Hook useEffect has missing dependencies: 'projects.features' and 'setProject'. Either include them or remove the dependency array
lint: src/components/map/MagicWand.js#L58
React Hook useEffect has missing dependencies: 'activeClass.color', 'activeClass.name', 'activeProject.properties.name', 'dispatchSetItems', 'items', 'map', and 'wand'. Either include them or remove the dependency array
lint: src/components/map/ProjectLayer.js#L109
React Hook useEffect has missing dependencies: 'dispatchSetPointsSelector' and 'map'. Either include them or remove the dependency array
lint: src/components/map/ProjectLayer.js#L119
React Hook useEffect has a missing dependency: 'map'. Either include it or remove the dependency array