Skip to content

docs(modal): Rails Modal Documentation #3830

docs(modal): Rails Modal Documentation

docs(modal): Rails Modal Documentation #3830

Triggered via pull request October 25, 2023 16:50
Status Success
Total duration 3m 23s
Artifacts

lint-test-build.yml

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

Annotations

11 warnings
Lint, Test, Build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
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