Skip to content

chore: revert v2 of pill (#1042) #2547

chore: revert v2 of pill (#1042)

chore: revert v2 of pill (#1042) #2547

Triggered via push August 12, 2024 15:34
Status Success
Total duration 3m 47s
Artifacts

test.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Run Unit Tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3, codecov/codecov-action@v4-beta. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Run Unit Tests
Codecov: Failed to properly create commit: The process '/home/runner/work/_actions/codecov/codecov-action/v4-beta/dist/codecov' failed with exit code 1
Run Unit Tests: malty/atoms/Floater/Floater.tsx#L68
React Hook useEffect has a missing dependency: 'handleScroll'. 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/ProductsBar/ProductsBar.tsx#L58
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#L94
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
Run Unit Tests: malty/molecules/Toast/Toast.tsx#L47
React Hook useEffect has missing dependencies: 'autoHideTimer', 'hideToast', and 'setAutoHideTimer'. Either include them or remove the dependency array