feat(deviceDetector): ACT-760 send device info to amplitude #356
release.yml
on: push
Build, Deploy to GitHub pages, Release npm package
0s
Annotations
2 errors and 11 warnings
Tests
Canceling since a higher priority waiting request for 'refs/heads/feature/ACT-760-device-performance-to-amplitude' exists
|
Tests
The operation was canceled.
|
Linting
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Linting:
src/components/Avatar/Avatar.stories.tsx#L103
React Hook useEffect has unnecessary dependencies: 'initialValue' and 'maxValue'. Either exclude them or remove the dependency array. Outer scope values like 'initialValue' aren't valid dependencies because mutating them doesn't re-render the component
|
Linting:
src/components/Avatar/Examples.stories.tsx#L165
The story should use PascalCase notation: environmentModel
|
Linting:
src/components/Lights/Lights.component.tsx#L58
React Hook useEffect has missing dependencies: 'lightTarget', 'scene', 'targets.head', and 'targets.shoe'. Either include them or remove the dependency array
|
Linting:
src/components/Models/BoundsModel/BoundsModel.container.tsx#L22
React Hook useCallback has a missing dependency: 'onLoaded'. Either include it or remove the dependency array
|
Linting:
src/components/Models/BoundsModel/BoundsModel.container.tsx#L29
React Hook useMemo has an unnecessary dependency: 'modelSrc'. Either exclude it or remove the dependency array
|
Linting:
src/components/Models/BoundsModel/BoundsModel.container.tsx#L36
React Hook useEffect has a missing dependency: 'bounds'. Either include it or remove the dependency array
|
Linting:
src/components/Models/Model/Model.component.tsx#L95
React Hook useMemo has a missing dependency: 'scene'. Either include it or remove the dependency array
|
Linting:
src/hooks/useDeviceDetector/use-device-detector.hook.ts#L100
Unexpected console statement
|
Linting:
src/hooks/useDeviceDetector/use-device-detector.hook.ts#L106
React Hook useEffect has a missing dependency: 'options'. Either include it or remove the dependency array. If 'setDeviceDetector' needs the current value of 'options', you can also switch to useReducer instead of useState and read 'options' in the reducer
|
Linting:
src/services/Analytics.service.ts#L7
Unexpected console statement
|