Skip to content

feat: add inline notification component #676

feat: add inline notification component

feat: add inline notification component #676

Triggered via pull request July 29, 2024 20:55
Status Success
Total duration 1m 31s
Artifacts

build.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

3 warnings
lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
lint: sdk/react/src/lib/components/settings-context-pane.tsx#L211
React Hook useEffect has a missing dependency: 'setExternalSettings'. Either include it or remove the dependency array. If 'setExternalSettings' changes too often, find the parent component that defines it and wrap that definition in useCallback
build
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/