post review #61
Annotations
1 error and 13 warnings
|
FULL → Build
Cannot invoke "org.kie.workbench.common.stunner.core.client.canvas.AbstractCanvas.disableHandlers()" because "canvas" is null
|
|
Setup environment
Restore cache failed: Dependencies file is not found in /Users/runner/work/kie-tools/kie-tools. Supported file pattern: go.sum
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/github-script@v6, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
FULL → Build:
packages/envelope/src/embedded/EmbeddedEnvelopeFactory.tsx#L96
React Hook useMemo has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useMemo call and refer to those specific props inside useMemo
|
|
FULL → Build:
packages/envelope/src/embedded/EmbeddedEnvelopeFactory.tsx#L101
React Hook useImperativeHandle has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useImperativeHandle call and refer to those specific props inside useImperativeHandle
|
|
FULL → Build:
packages/editor/src/embedded/embedded/EmbeddedEditor.tsx#L87
React Hook useMemo has an unnecessary dependency: 'props.file.getFileContents'. Either exclude it or remove the dependency array
|
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L66
React Hook useMemo has missing dependencies: 'envelopeContext.operatingSystem' and 'envelopeContext.services.keyboardShortcuts'. Either include them or remove the dependency array
|
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L66
React Hook useMemo has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked
|
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L76
React Hook useEffect has a missing dependency: 'envelopeContext.services.keyboardShortcuts'. Either include it or remove the dependency array
|
|
FULL → Build:
packages/editor/src/envelope/KeyBindingsHelpOverlay/KeyBindingsHelpOverlay.tsx#L85
React Hook useEffect has a missing dependency: 'envelopeContext.services.keyboardShortcuts'. Either include it or remove the dependency array
|
|
FULL → Build:
packages/dashbuilder-component-svg-heatmap/src/SVGHeatmapComponent.tsx#L139
React Hook useEffect has missing dependencies: 'onDataset' and 'props.controller'. Either include them or remove the dependency array
|
|
FULL → Build:
packages/dashbuilder-component-svg-heatmap/src/SvgHeatmap.tsx#L93
React Hook useEffect has a missing dependency: 'parentRef'. Either include it or remove the dependency array
|
|
FULL → Build:
packages/dashbuilder-component-svg-heatmap/src/SvgHeatmap.tsx#L119
React Hook useEffect has a missing dependency: 'props.containsId'. Either include it or remove the dependency array
|
|
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "Linux_0__build-artifacts", "Linux_0__end-to-end-tests-artifacts", "Linux_0__end-to-end-tests-reports", "Linux_0__tests-reports", "Linux_1__build-artifacts", "Linux_1__end-to-end-tests-artifacts", "Linux_1__end-to-end-tests-reports", "Linux_1__tests-reports", "Linux_2__build-artifacts", "Linux_2__end-to-end-tests-artifacts", "Linux_2__end-to-end-tests-reports", "Linux_2__tests-reports", "macOS_0__build-artifacts", "macOS_0__end-to-end-tests-artifacts", "macOS_0__end-to-end-tests-reports", "macOS_0__tests-reports", "macOS_1__build-artifacts", "macOS_1__end-to-end-tests-artifacts", "macOS_1__end-to-end-tests-reports", "macOS_1__tests-reports", "macOS_2__build-artifacts", "macOS_2__end-to-end-tests-artifacts", "macOS_2__end-to-end-tests-reports", "macOS_2__tests-reports", "Windows_0__build-artifacts", "Windows_0__end-to-end-tests-artifacts", "Windows_0__end-to-end-tests-reports", "Windows_0__tests-reports", "Windows_1__build-artifacts", "Windows_1__end-to-end-tests-artifacts", "Windows_1__end-to-end-tests-reports", "Windows_1__tests-reports", "Windows_2__build-artifacts", "Windows_2__end-to-end-tests-artifacts", "Windows_2__end-to-end-tests-reports", "Windows_2__tests-reports".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
Loading