Merge pull request #77 from ZackGoldblum/starfield-revamp #58
ZackGoldblumAnnotations
6 errors and 1 warning
|
deploy
Process completed with exit code 1.
|
|
deploy:
src/components/Starfield.jsx#L379
'index' is defined but never used
|
|
deploy:
src/components/Starfield.jsx#L370
'scrollProgress' is assigned a value but never used
|
|
deploy:
src/components/Starfield.jsx#L13
'skyboxLoaded' is assigned a value but never used
|
|
deploy:
src/components/Starfield.jsx#L4
'uiVisible' is missing in props validation
|
|
deploy:
src/components/Starfield.jsx#L4
'onSkyboxLoaded' is missing in props validation
|
|
deploy:
src/components/Starfield.jsx#L449
React Hook useEffect has a missing dependency: 'onSkyboxLoaded'. Either include it or remove the dependency array. If 'onSkyboxLoaded' changes too often, find the parent component that defines it and wrap that definition in useCallback
|