Skip to content

feat: add inline notification component #681

feat: add inline notification component

feat: add inline notification component #681

Triggered via pull request August 1, 2024 21:06
Status Success
Total duration 1m 32s
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/