Skip to content

[DSS-328] Upload Card - Layout Update #3750

[DSS-328] Upload Card - Layout Update

[DSS-328] Upload Card - Layout Update #3750

Triggered via pull request September 19, 2023 15:13
Status Success
Total duration 4m 1s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

lint-test-build.yml

on: pull_request
Lint, Test, Build
3m 50s
Lint, Test, Build
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Lint, Test, Build: packages/sage-react/lib/Drawer/Drawer.jsx#L43
React Hook useEffect has a missing dependency: 'onExpandChange'. Either include it or remove the dependency array. If 'onExpandChange' changes too often, find the parent component that defines it and wrap that definition in useCallback
Lint, Test, Build: packages/sage-react/lib/Input/Input.jsx#L106
React Hook useEffect has a missing dependency: 'inputStyles'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/PanelControls/PanelControls.story.jsx#L150
React Hook useEffect has a missing dependency: 'fetchData'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Table/Table.jsx#L176
React Hook useEffect has a missing dependency: 'buildHeaders'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Table/Table.jsx#L182
React Hook useEffect has a missing dependency: 'getSelectionType'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Table/TableRow.jsx#L79
React Hook useEffect has a missing dependency: 'hasBorders'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Toast/Toast.jsx#L45
React Hook useEffect has a missing dependency: 'dismiss'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Toast/Toast.jsx#L50
React Hook useEffect has a missing dependency: 'dismiss'. Either include it or remove the dependency array
Lint, Test, Build: packages/sage-react/lib/Toggle/Switch.story.jsx#L106
Using propTypes from another component is not safe because they may be removed in production builds
Lint, Test, Build: packages/sage-react/lib/Tooltip/TooltipElement.jsx#L74
React Hook useLayoutEffect has missing dependencies: 'parentDomRect' and 'position'. Either include them or remove the dependency array. Mutable values like 'tooltipRef.current' aren't valid dependencies because mutating them doesn't re-render the component