Skip to content

Merge pull request #515 from inokawa/eslint-v9 #2196

Merge pull request #515 from inokawa/eslint-v9

Merge pull request #515 from inokawa/eslint-v9 #2196

Triggered via push September 21, 2024 04:26
Status Success
Total duration 9m 23s
Artifacts

check.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
check: src/react/VGrid.tsx#L319
React Hook useImperativeHandle has missing dependencies: 'hStore', 'scroller._scrollBy', 'scroller._scrollTo', 'scroller._scrollToIndex', and 'vStore'. Either include them or remove the dependency array
check: src/react/Virtualizer.tsx#L315
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/Virtualizer.tsx#L339
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/WindowVirtualizer.tsx#L219
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/WindowVirtualizer.tsx#L230
React Hook useImperativeHandle has a missing dependency: 'store'. Either include it or remove the dependency array
check: src/solid/ListItem.tsx#L32
The reactive variable 'props' should not be reassigned or altered directly
check: src/solid/ListItem.tsx#L32
The reactive variable 'props' should not be reassigned or altered directly
check: src/solid/RangedFor.tsx#L40
For proper analysis, a variable should be used to capture the result of this function call
check: src/solid/RangedFor.tsx#L53
For proper analysis, array destructuring should be used to capture the first result of this function call
check: src/solid/VList.tsx#L53
The reactive variable 'props' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function, or else changes will be ignored