Skip to content

fix: removes dasherize from variable used for attributes #4084

fix: removes dasherize from variable used for attributes

fix: removes dasherize from variable used for attributes #4084

Triggered via pull request June 3, 2024 17:51
Status Success
Total duration 2m 49s
Artifacts

lint-test-build.yml

on: pull_request
Matrix: Lint, Test, Build
Fit to window
Zoom out
Zoom in

Annotations

30 warnings
Lint, Test, Build (20): 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 (20): 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 (20): 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 (20): 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 (20): 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 (20): 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 (20): 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 (20): 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 (20): 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 (16): 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 (16): 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 (16): 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 (16): 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 (16): 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 (16): 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 (16): 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 (16): 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 (16): 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 (18): 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 (18): 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 (18): 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 (18): 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 (18): 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 (18): 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 (18): 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 (18): 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 (18): 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