Avatar: update minor documentation #3662
monicawheelerlint-test-build.yml
on: pull_request
Lint, Test, Build
4m 33s
Annotations
11 warnings
|
Lint, Test, Build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
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
|