Skip to content

Merge pull request #63 from davisshaver/bump-deps #171

Merge pull request #63 from davisshaver/bump-deps

Merge pull request #63 from davisshaver/bump-deps #171

Triggered via push June 3, 2024 15:17
Status Success
Total duration 1m 47s
Artifacts

lint.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

5 warnings
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
lint: src/connect.jsx#L156
React Hook React.useCallback has missing dependencies: 'mockLogin', 'onError', 'onLogin', 'provider', 'redirectBoomerang', 'redirectURL', and 'signMessageAsync'. Either include them or remove the dependency array. If 'onLogin' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/connect.jsx#L216
React Hook React.useEffect has missing dependencies: 'activeConnector', 'connections', 'disconnect', 'onLogout', 'signIn', and 'triggeredLogin'. Either include them or remove the dependency array. If 'onLogout' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/settings.jsx#L162
React Hook useEffect has missing dependencies: 'checkboxes', 'setValue', and 'state'. Either include them or remove the dependency array. You can also do a functional update 'setState(s => ...)' if you only need 'state' in the 'setState' call