Skip to content

Merge pull request #247 from boostcampwm-2024/refactor/front-#241 #409

Merge pull request #247 from boostcampwm-2024/refactor/front-#241

Merge pull request #247 from boostcampwm-2024/refactor/front-#241 #409

Triggered via push December 2, 2024 12:00
Status Success
Total duration 43s
Artifacts

CI.yml

on: push
BE-test-and-build
34s
FE-test-and-build
17s

Annotations

2 warnings
FE-test-and-build: FE/src/components/StocksDetail/Chart.tsx#L83
React Hook useEffect has missing dependencies: 'chartX', 'lowerChartCanvasRef', 'lowerChartY', 'resizeCanvases', 'upperChartCanvasRef', and 'upperChartY'. Either include them or remove the dependency array. Outer scope values like 'setCanvasSize' aren't valid dependencies because mutating them doesn't re-render the component
FE-test-and-build: FE/src/hooks/useCanvasResize.ts#L54
React Hook useCallback has missing dependencies: 'canvasRefs.chartX', 'canvasRefs.lowerChartCanvasRef', 'canvasRefs.lowerChartY', 'canvasRefs.upperChartCanvasRef', and 'canvasRefs.upperChartY'. Either include them or remove the dependency array