Skip to content

task/WG-401-Add-Asset-to-Feature-Hook #846

task/WG-401-Add-Asset-to-Feature-Hook

task/WG-401-Add-Asset-to-Feature-Hook #846

Triggered via pull request January 28, 2025 21:50
Status Failure
Total duration 2m 4s
Artifacts

main.yml

on: pull_request
Angular-Unit-Tests
55s
Angular-Unit-Tests
Angular-Linting
49s
Angular-Linting
React-Unit-Tests
1m 54s
React-Unit-Tests
React-Linting
32s
React-Linting
React-Build
49s
React-Build
Fit to window
Zoom out
Zoom in

Annotations

1 error and 8 warnings
React-Unit-Tests
Process completed with exit code 1.
React-Linting: react/src/components/Files/FileListing.tsx#L74
React Hook useEffect has a missing dependency: 'selectedSystem'. Either include it or remove the dependency array
React-Linting: react/src/components/Files/FileListing.tsx#L92
React Hook useEffect has a missing dependency: 'allowedFileExtensions'. Either include it or remove the dependency array. If 'setChonkyFiles' needs the current value of 'allowedFileExtensions', you can also switch to useReducer instead of useState and read 'allowedFileExtensions' in the reducer
React-Linting: react/src/components/Files/FileListing.tsx#L163
React Hook useCallback has missing dependencies: 'onFileSelect', 'onFolderSelect', and 'selectedFiles'. Either include them or remove the dependency array. If 'onFolderSelect' changes too often, find the parent component that defines it and wrap that definition in useCallback
React-Linting: react/src/components/Systems/SystemSelect.tsx#L33
React Hook useEffect has a missing dependency: 'onSystemSelect'. Either include it or remove the dependency array. If 'onSystemSelect' changes too often, find the parent component that defines it and wrap that definition in useCallback
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 22) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 28) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 34) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/pages/Login/Login.tsx#L29
React Hook useEffect has missing dependencies: 'configuration.geoapiUrl', 'isAuthenticated', 'location.search', and 'navigate'. Either include them or remove the dependency array