Skip to content
Node.js CI

Move event dispatch location for more consistent state #2126

Sign in to view logs

Move event dispatch location for more consistent state

Move event dispatch location for more consistent state #2126

Triggered via pull request January 23, 2025 10:36
@gkjohnsongkjohnson
opened #942
load-event-improvements
Status Success
Total duration 46s
Artifacts –
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

node.js.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

6 warnings
build (20.x)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build (20.x): src/r3f/components/CameraTransition.jsx#L42
React Hook useMemo has missing dependencies: 'camera' and 'mode'. Either include them or remove the dependency array
build (20.x): src/r3f/components/CameraTransition.jsx#L150
React Hook useEffect has a missing dependency: 'onBeforeToggle'. Either include it or remove the dependency array
build (18.x)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build (18.x): src/r3f/components/CameraTransition.jsx#L42
React Hook useMemo has missing dependencies: 'camera' and 'mode'. Either include them or remove the dependency array
build (18.x): src/r3f/components/CameraTransition.jsx#L150
React Hook useEffect has a missing dependency: 'onBeforeToggle'. Either include it or remove the dependency array