Skip to content

app: useEffect warning fixes #621

app: useEffect warning fixes

app: useEffect warning fixes #621

Triggered via pull request January 8, 2024 09:37
Status Success
Total duration 3m 23s
Artifacts

dev-checks.yml

on: pull_request
Matrix: tests
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
tests (16.x): src/Components/CreateImageWizard/formComponents/MountPoint.js#L53
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
tests (16.x): src/Components/CreateImageWizard/formComponents/Packages.js#L255
React Hook useEffect has a missing dependency: 'keydownHandler'. Either include it or remove the dependency array
tests (16.x): src/Components/CreateImageWizard/formComponents/Repositories.js#L256
React Hook useMemo has missing dependencies: 'data' and 'initializeRepositories'. Either include them or remove the dependency array
tests (16.x): src/Components/CreateImageWizard/formComponents/Repositories.js#L292
React Hook useMemo has a missing dependency: 'isRepoSelected'. Either include it or remove the dependency array
tests (16.x): src/Components/CreateImageWizard/formComponents/SizeUnit.js#L20
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
tests (18.x): src/Components/CreateImageWizard/formComponents/MountPoint.js#L53
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
tests (18.x): src/Components/CreateImageWizard/formComponents/Packages.js#L255
React Hook useEffect has a missing dependency: 'keydownHandler'. Either include it or remove the dependency array
tests (18.x): src/Components/CreateImageWizard/formComponents/Repositories.js#L256
React Hook useMemo has missing dependencies: 'data' and 'initializeRepositories'. Either include them or remove the dependency array
tests (18.x): src/Components/CreateImageWizard/formComponents/Repositories.js#L292
React Hook useMemo has a missing dependency: 'isRepoSelected'. Either include it or remove the dependency array
tests (18.x): src/Components/CreateImageWizard/formComponents/SizeUnit.js#L20
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect