Skip to content

Merge pull request #251 from boostcampwm-2024/refactor/socket/unsubsc… #422

Merge pull request #251 from boostcampwm-2024/refactor/socket/unsubsc…

Merge pull request #251 from boostcampwm-2024/refactor/socket/unsubsc… #422

Triggered via push December 3, 2024 01:44
Status Success
Total duration 54s
Artifacts

CI.yml

on: push
BE-test-and-build
45s
BE-test-and-build
FE-test-and-build
18s
FE-test-and-build
Fit to window
Zoom out
Zoom in

Annotations

5 warnings
FE-test-and-build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
FE-test-and-build: FE/src/components/Search/index.tsx#L39
React Hook useEffect has missing dependencies: 'addSearchHistory' and 'isLoading'. Either include them or remove the dependency array
FE-test-and-build: FE/src/components/StocksDetail/Chart.tsx#L291
React Hook useCallback has unnecessary dependencies: 'drawBarChart', 'drawCandleChart', 'drawChartGrid', 'drawLineChart', 'drawLowerYAxis', 'drawUpperYAxis', 'drawXAxis', and 'padding'. Either exclude them or remove the dependency array. Outer scope values like 'padding' aren't valid dependencies because mutating them doesn't re-render the component
FE-test-and-build: FE/src/components/StocksDetail/PriceSection.tsx#L34
React Hook useCallback has a missing dependency: 'queryClient'. Either include it or remove the dependency array
BE-test-and-build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636