Add ability to edit/delete reviews, refactor review components #278
lint.yml
on: pull_request
Lint and check formatting
36s
Annotations
10 warnings
Lint and check formatting:
site/src/component/GradeDist/GradeDist.tsx#L70
React Hook useEffect has a missing dependency: 'fetchGradeDistData'. Either include it or remove the dependency array
|
Lint and check formatting:
site/src/component/GradeDist/GradeDist.tsx#L81
React Hook useEffect has missing dependencies: 'createCourseEntries', 'createProfEntries', 'props.course', and 'props.professor'. Either include them or remove the dependency array
|
Lint and check formatting:
site/src/component/GradeDist/GradeDist.tsx#L88
React Hook useEffect has missing dependencies: 'createQuarterEntries' and 'gradeDistData.length'. Either include them or remove the dependency array
|
Lint and check formatting:
site/src/component/Report/ReportGroup.tsx#L21
Unexpected console statement
|
Lint and check formatting:
site/src/component/Report/ReportGroup.tsx#L27
React Hook useEffect has a missing dependency: 'props.reviewID'. Either include it or remove the dependency array
|
Lint and check formatting:
site/src/component/Report/Reports.tsx#L47
React Hook useEffect has a missing dependency: 'getData'. Either include it or remove the dependency array
|
Lint and check formatting:
site/src/component/Review/Review.tsx#L53
React Hook useEffect has missing dependencies: 'dispatch' and 'getReviews'. Either include them or remove the dependency array
|
Lint and check formatting:
site/src/component/ReviewForm/ReviewForm.tsx#L108
React Hook useEffect has missing dependencies: 'cookies.user' and 'props'. Either include them 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
|
Lint and check formatting:
site/src/component/Schedule/Schedule.tsx#L41
React Hook useEffect has a missing dependency: 'fetchScheduleDataFromAPI'. Either include it or remove the dependency array
|
Lint and check formatting:
site/src/component/SideInfo/SideInfo.tsx#L138
React Hook useEffect has missing dependencies: 'allToken' and 'props.searchType'. Either include them or remove the dependency array
|