feat(icons): update close icons to use the rh micron #12283
Merged
Annotations
11 warnings
|
Complete job
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/cache@v4, actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Run linter:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L145
React Hook useEffect has a missing dependency: 'dateParse'. Either include it or remove the dependency array. If 'dateParse' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
|
Run linter:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L134
React Hook useMemo has a missing dependency: 'placeholder.length'. Either include it or remove the dependency array
|
|
Run linter:
packages/react-core/src/components/Card/examples/CardWithModifiers.tsx#L18
Mutating a value returned from 'useState()', which should not be mutated. Use the setter function to update instead
|
|
Run linter:
packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L191
React Hook useEffect has a missing dependency: 'shouldFocus'. Either include it or remove the dependency array
|
|
Run linter:
packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L184
React Hook useEffect has a missing dependency: 'focusedDate'. Either include it or remove the dependency array
|
|
Run linter:
packages/react-core/src/components/BackToTop/BackToTop.tsx#L36
The 'toggleVisible' function makes the dependencies of useEffect Hook (at line 74) change on every render. Move it inside the useEffect callback. Alternatively, wrap the definition of 'toggleVisible' in its own useCallback() Hook
|
|
Run linter:
packages/react-core/src/components/Alert/AlertGroup.tsx#L70
React Hook useEffect has a missing dependency: 'getTargetElement'. Either include it or remove the dependency array
|
|
Run linter:
packages/react-core/src/components/Alert/Alert.tsx#L167
React Hook useEffect has missing dependencies: 'hasAnimations', 'offstageRight', and 'updateTransitionEnd'. Either include them or remove the dependency array
|
|
Run linter:
packages/react-charts/src/victory/components/ChartTooltip/examples/ChartTooltipLegend.tsx#L14
Hooks must be called at the top level in the body of a function component or custom hook, and may not be called within function expressions. See the Rules of Hooks (https://react.dev/warnings/invalid-hook-call-warning)
|
|
Run linter:
packages/react-charts/src/victory/components/ChartLegend/examples/ChartLegendTooltips.tsx#L20
Hooks must be called at the top level in the body of a function component or custom hook, and may not be called within function expressions. See the Rules of Hooks (https://react.dev/warnings/invalid-hook-call-warning)
|
Loading