Skip to content

Refactor range computation #1017

Refactor range computation

Refactor range computation #1017

Triggered via pull request October 1, 2023 04:04
Status Success
Total duration 7m 6s
Artifacts

check.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
check: src/react/ListItem.tsx#L87
React Hook useMemo has missing dependencies: 'isHorizontal' and 'isRtl'. Either include them or remove the dependency array
check: src/react/VGrid.tsx#L142
React Hook useMemo has a missing dependency: 'isRtl'. Either include it or remove the dependency array
check: src/react/VGrid.tsx#L407
React Hook useImperativeHandle has missing dependencies: 'hScroller', 'hStore', 'vScroller', and 'vStore'. Either include them or remove the dependency array
check: src/react/VGrid.tsx#L468
React Hook useMemo has missing dependencies: 'ItemElement', 'hStore', 'isRtl', 'resizer', and 'vStore'. Either include them or remove the dependency array
check: src/react/VGrid.tsx#L496
React Hook useMemo was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies
check: src/react/VGrid.tsx#L496
React Hook useMemo has a missing dependency: 'viewportAttrs'. Either include it or remove the dependency array
check: src/react/VList.tsx#L278
React Hook useEffect has a missing dependency: 'onScrollStop'. Either include it or remove the dependency array
check: src/react/VList.tsx#L284
React Hook useEffect has a missing dependency: 'onRangeChangeProp'. Either include it or remove the dependency array. If 'onRangeChangeProp' changes too often, find the parent component that defines it and wrap that definition in useCallback
check: src/react/VList.tsx#L307
React Hook useImperativeHandle has missing dependencies: 'scroller._scrollBy', 'scroller._scrollTo', 'scroller._scrollToIndex', and 'store'. Either include them or remove the dependency array
check: src/react/VList.tsx#L340
React Hook useMemo has missing dependencies: 'ItemElement', 'isHorizontal', 'isRtl', 'resizer', and 'store'. Either include them or remove the dependency array