feat(link): [DSM-715] improve flexibility #1661
Triggered via pull request
September 19, 2023 08:46
acpcor
Status
Success
Total duration
12m 54s
Artifacts
–
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
10 warnings
|
Run Unit Tests:
malty/atoms/Floater/Floater.tsx#L66
React Hook useEffect has a missing dependency: 'handleScroll'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/atoms/Icon/Icons.MultipleIcons.stories.tsx#L94
React Hook useLayoutEffect has a missing dependency: 'params'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/atoms/Icon/Icons.SingleIcon.stories.tsx#L97
React Hook useLayoutEffect has a missing dependency: 'params'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/atoms/Icon/Icons.SingleIcon.stories.tsx#L108
React Hook useEffect has a missing dependency: 'params'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/atoms/Tooltip/Tooltip.helper.tsx#L93
React Hook useEffect has a missing dependency: 'tooltipId'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/atoms/Tooltip/Tooltip.helper.tsx#L137
React Hook useEffect has a missing dependency: 'isOpen'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/molecules/AlertBanner/AlertBanner.helper.ts#L54
Assignments to the 'throttleTimeout' variable from inside React Hook useLayoutEffect will be lost after each render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside useLayoutEffect
|
|
Run Unit Tests:
malty/molecules/NavList/NavList.tsx#L74
React Hook useEffect has a missing dependency: 'setInitialActiveItem'. Either include it or remove the dependency array
|
|
Run Unit Tests:
malty/molecules/ProductsBar/ProductsBar.tsx#L56
React Hook useEffect has missing dependencies: 'ref' and 'setProfileMenuOpen'. Either include them or remove the dependency array. If 'setProfileMenuOpen' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
|
Run Unit Tests:
malty/molecules/Stepper/Stepper.tsx#L97
React Hook useEffect has missing dependencies: 'dataQaId', 'isMultiStep', and 'theme'. Either include them or remove the dependency array. If 'setElHtml' needs the current value of 'dataQaId', you can also switch to useReducer instead of useState and read 'dataQaId' in the reducer
|