Material UI: Study programme overview - Update statistics tab #454
pull_request.yaml
on: pull_request
Lint backend and frontend
1m 44s
Matrix: test_end_to_end
Annotations
10 warnings
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L39
React Hook useMemo has a missing dependency: 'selectedSemesters'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/FacultiesTab/index.jsx#L65
React Hook useMemo has a missing dependency: 'totalRow'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/index.jsx#L51
React Hook useEffect has a missing dependency: 'semesterFilter'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/ColorizedCoursesTable/selectorComponents.jsx#L76
React Hook useMemo has a missing dependency: 'getTextIn'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L37
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L44
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L51
React Hook useEffect has a missing dependency: 'searchList'. Either include it or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/CustomOpenUniSearch.jsx#L84
React Hook useEffect has missing dependencies: 'parseQueryFromUrl' and 'setValues'. Either include them or remove the dependency array. If 'setValues' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
Lint backend and frontend:
services/frontend/src/components/CustomOpenUniPopulation/OpenUniPopulationResults/index.jsx#L232
React Hook useEffect has missing dependencies: 'getTextIn', 'isError', and 'isFetchingOrLoading'. Either include them or remove the dependency array
|
Lint backend and frontend:
services/frontend/src/components/CustomPopulation/index.jsx#L44
The 'allSemesters' logical expression could make the dependencies of useMemo Hook (at line 84) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'allSemesters' in its own useMemo() Hook
|