Docs: Icon documentation update (Rails) #3769
pixelflipslint-test-build.yml
on: pull_request
Lint, Test, Build
3m 33s
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/Input/Input.story.jsx#L154
Unexpected console statement
|
|
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
|