Update all non-major dependencies #405
pull_request.yaml
on: pull_request
Lint backend and frontend
1m 32s
Matrix: test_end_to_end
Annotations
3 errors and 10 warnings
Lint backend and frontend:
services/frontend/src/components/material/FacultyGraduations/GraduationTimes/MedianDisplay/MedianBarChart.tsx#L162
Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read
|
Lint backend and frontend:
services/frontend/src/components/material/FacultyGraduations/GraduationTimes/MedianDisplay/MedianBarChart.tsx#L198
Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read
|
Lint backend and frontend
Process completed with exit code 1.
|
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
|