Skip to content

feat(react-charts): add high contrast #4128

feat(react-charts): add high contrast

feat(react-charts): add high contrast #4128

Triggered via pull request May 8, 2026 06:05
Status Failure
Total duration 1m 46s
Artifacts

main.yml

on: pull_request
Build demo app
0s
Build demo app
Unit tests
0s
Unit tests
Matrix: Integration tests
Fit to window
Zoom out
Zoom in

Annotations

7 errors and 12 warnings
Build
Process completed with exit code 2.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L121
Cannot find module '@patternfly/react-tokens/dist/esm/chart_bullet_bar_stroke_Width' or its corresponding type declarations.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L120
Cannot find module '@patternfly/react-tokens/dist/esm/chart_bullet_bar_stroke_Color' or its corresponding type declarations.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L105
Cannot find module '@patternfly/react-tokens/dist/esm/chart_donut_pie_data_stroke_Width' or its corresponding type declarations.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L104
Cannot find module '@patternfly/react-tokens/dist/esm/chart_donut_pie_data_stroke_Color' or its corresponding type declarations.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L61
Cannot find module '@patternfly/react-tokens/dist/esm/chart_legend_data_stroke_Width' or its corresponding type declarations.
Build: packages/react-charts/src/victory/components/ChartTheme/themes/base-theme.ts#L60
Cannot find module '@patternfly/react-tokens/dist/esm/chart_legend_data_stroke_Color' or its corresponding type declarations.
Build
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/
Lint
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/
Lint: 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
Lint: 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
Lint: 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
Lint: packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L199
React Hook useEffect has a missing dependency: 'shouldFocus'. Either include it or remove the dependency array
Lint: packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L192
React Hook useEffect has a missing dependency: 'focusedDate'. Either include it or remove the dependency array
Lint: 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
Lint: 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
Lint: 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
Lint: 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)
Lint: 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)