Skip to content

refactor: remove default prop usage across components #747

refactor: remove default prop usage across components

refactor: remove default prop usage across components #747

Triggered via pull request December 19, 2024 15:20
Status Success
Total duration 4m 23s
Artifacts

validate-and-build.yaml

on: pull_request
validate-and-build
4m 16s
validate-and-build
Fit to window
Zoom out
Zoom in

Annotations

7 warnings
validate-and-build: packages/example/gatsby-ssr.js#L24
Dangerous property 'dangerouslySetInnerHTML' found
validate-and-build: packages/gatsby-theme-carbon/src/components/LeftNav/LeftNav.js#L57
React Hook useEffect has a missing dependency: 'hasNestedLevels'. Either include it or remove the dependency array
validate-and-build: packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L32
'i' is defined but never used. Allowed unused args must match /res|next|^err|^_/u
validate-and-build: packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L188
'prevProps' is defined but never used. Allowed unused args must match /res|next|^err|^_/u
validate-and-build: packages/gatsby-theme-carbon/src/components/Tabs/Tabs.js#L141
React Hook useMemo has an unnecessary dependency: 'tabList.current'. Either exclude it or remove the dependency array. Mutable values like 'tabList.current' aren't valid dependencies because mutating them doesn't re-render the component
validate-and-build: packages/gatsby-theme-carbon/src/util/context/NavContext.js#L35
The 'toggleNavState' function makes the dependencies of useMemo Hook (at line 54) change on every render. Move it inside the useMemo callback. Alternatively, wrap the definition of 'toggleNavState' in its own useCallback() Hook