Skip to content

app: useEffect warning fixes #641

app: useEffect warning fixes

app: useEffect warning fixes #641

Triggered via pull request January 9, 2024 10:35
Status Success
Total duration 3m 22s
Artifacts

dev-checks.yml

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

Annotations

12 warnings
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#L162
React Hook useEffect has a missing dependency: 'selectedPackages'. Either include it or remove the dependency array
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
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#L162
React Hook useEffect has a missing dependency: 'selectedPackages'. Either include it or remove the dependency array
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