Skip to content

fix: useAsync dependency arrays #4733

fix: useAsync dependency arrays

fix: useAsync dependency arrays #4733

Triggered via pull request July 13, 2023 16:12
@iamacookiamacook
opened #2291
Status Success
Total duration 16s
Artifacts

cla.yml

on: pull_request_target
CLAAssistant
7s
CLAAssistant
Fit to window
Zoom out
Zoom in

Annotations

2 warnings
CLAAssistant
The following actions uses node12 which is deprecated and will be forced to run on node16: contributor-assistant/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
react-hooks/exhaustive-deps: src/components/tx/security/redefine/useRedefine.ts#L97
React Hook useAsync has an unnecessary dependency: 'retryCounter'. Either exclude it or remove the dependency array. * [SUGGESTION] Update the dependencies array to be: [safe.chainId, safe.threshold, safeAddress, safeTransaction, wallet.address, isFeatureEnabled]