Skip to content

Merge branch 'master' into v4.0.0 #8

Merge branch 'master' into v4.0.0

Merge branch 'master' into v4.0.0 #8

Triggered via push June 28, 2024 19:11
Status Failure
Total duration 1m 13s
Artifacts

workflow.yml

on: push
Prepare environment
59s
Prepare environment
Release
0s
Release
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 9 warnings
Run tests: src/components/mini-map/mini-map.tsx#L186
Optional chain expressions can return undefined by design - using a non-null assertion is unsafe and wrong
Run tests: src/components/mini-map/mini-map.tsx#L187
Optional chain expressions can return undefined by design - using a non-null assertion is unsafe and wrong
Run tests: src/core/instance.core.ts#L20
'isExcludedNode' is defined but never used
Run tests: src/core/instance.core.ts#L486
Unexpected console statement
Run tests: src/core/pan/velocity.logic.ts#L18
Unexpected use of 'isNaN'. Use Number.isNaN instead https://github.com/airbnb/javascript#standard-library--isnan
Run tests: src/core/pan/velocity.logic.ts#L32
Unexpected use of 'isNaN'. Use Number.isNaN instead https://github.com/airbnb/javascript#standard-library--isnan
Run tests: src/core/pinch/pinch.logic.ts#L21
Unary operator '++' used
Run tests: src/core/pinch/pinch.logic.ts#L87
Expected '===' and instead saw '=='
Run tests: src/core/pinch/pinch.logic.ts#L87
Expected '===' and instead saw '=='
Run tests: src/utils/callback.utils.ts#L2
'ReactZoomPanPinchContext' is defined but never used
Prepare environment
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache/save@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Prepare environment
Cache save failed.
Run tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache/restore@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Run tests: src/components/transform-component/transform-component.tsx#L43
React Hook useEffect has missing dependencies: 'cleanupWindowEvents' and 'init'. Either include them or remove the dependency array
Run tests: src/core/animations/animations.constants.ts#L55
Unexpected unnamed function
Run tests: src/core/animations/animations.constants.ts#L59
Unexpected unnamed function
Run tests: src/hooks/use-transform-init.tsx#L26
React Hook useEffect has missing dependencies: 'callback' and 'libraryContext'. Either include them or remove the dependency array. If 'callback' changes too often, find the parent component that defines it and wrap that definition in useCallback
Run tests: src/hooks/use-zoom-pan-pinch.tsx#L31
The ref value 'instance.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'instance.current' to a variable inside the effect, and use that variable in the cleanup function
Run tests: src/hooks/use-zoom-pan-pinch.tsx#L34
React Hook useLayoutEffect has unnecessary dependencies: 'contentRef.current' and 'wrapperRef.current'. Either exclude them or remove the dependency array. Mutable values like 'contentRef.current' aren't valid dependencies because mutating them doesn't re-render the component