Internal: Execute Custom Core tests from core repo [ED-15774] #46502
baghdasarovelementorAnnotations
11 warnings
|
Lint - File Diff
The following actions use a deprecated Node.js version and will be forced to run on node20: technote-space/get-diff-action@v6. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
JS-Lint:
app/assets/js/hooks/use-page-title.js#L6
Assignments to the 'prefix' variable from inside React Hook useEffect 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 useEffect
|
|
JS-Lint:
app/assets/js/molecules/collapse.js#L21
React Hook useEffect has a missing dependency: 'isOpened'. Either include it or remove the dependency array
|
|
JS-Lint:
app/assets/js/molecules/collapse.js#L27
React Hook useEffect 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 useEffect call and refer to those specific props inside useEffect
|
|
JS-Lint:
app/assets/js/molecules/tooltip.js#L70
React Hook useEffect has missing dependencies: 'isLibraryLoaded' and 'setTipsy'. Either include them or remove the dependency array
|
|
JS-Lint:
app/assets/js/molecules/tooltip.js#L76
React Hook useEffect has a missing dependency: 'setTipsy'. Either include it or remove the dependency array
|
|
JS-Lint:
app/assets/js/molecules/tooltip.js#L83
React Hook useEffect has missing dependencies: 'props.disabled' and 'showTooltip'. Either include them or remove the dependency array
|
|
JS-Lint:
app/assets/js/organisms/unfiltered-files-dialog.js#L32
React Hook useEffect has missing dependencies: 'onEnable', 'setAjax', and 'setShow'. Either include them or remove the dependency array
|
|
JS-Lint:
app/assets/js/organisms/unfiltered-files-dialog.js#L45
React Hook useEffect has missing dependencies: 'onReady' and 'setShow'. Either include them or remove the dependency array
|
|
JS-Lint:
app/assets/js/organisms/unfiltered-files-dialog.js#L51
React Hook useEffect has a missing dependency: 'onLoad'. Either include it or remove the dependency array
|
|
JS-Lint:
app/assets/js/ui/modal/modal.js#L103
React Hook useEffect has missing dependencies: 'closeModal' and 'props'. Either include them 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 useEffect call and refer to those specific props inside useEffect
|