Test

[Address suppressed ESLint errors] Fix exhaustive deps errors in App.… #12743

Sign in to view logs

Triggered via push January 17, 2025 19:37

joel-jeremy

master

Status Failure

Total duration 1m 16s

Artifacts –

check.yml

on: push

Annotations

1 error and 4 warnings

Process completed with exit code 1.

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

lint: packages/desktop-client/src/components/App.tsx#L149

React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array. Outer scope values like 'addNotification' aren't valid dependencies because mutating them doesn't re-render the component

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636