You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have strict tslint/eslint rules and noticed that the react-async-hook code produces these warnings.
Line 126: React Hook useCallback has a missing dependency: 'options'. Either include it or remove the dependency array. If 'setValue' needs the current value of 'options', you can also switch to useReducer instead of useState and read 'options' in the reducer react-hooks/exhaustive-deps
Line 132: React Hook useCallback has a missing dependency: 'options'. Either include it or remove the dependency array. If 'setValue' needs the current value of 'options', you can also switch to useReducer instead of useState and read 'options' in the reducer react-hooks/exhaustive-deps
Line 137: React Hook useCallback has a missing dependency: 'options'. Either include it or remove the dependency array. If 'setValue' needs the current value of 'options', you can also switch to useReducer instead of useState and read 'options' in the reducer react-hooks/exhaustive-deps
Line 249: React Hook useEffect was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies
react-hooks/exhaustive-deps
Line 249: React Hook useEffect has missing dependencies: 'executeAsyncOperation', 'isMounting', 'normalizedOptions.executeOnMount', 'normalizedOptions.executeOnUpdate', and 'params'. Either include them or remove the dependency array
react-hooks/exhaustive-deps
Is it save to include the dependencies that are missing or will that cause unexpected behaviour?
Thanks
/Martin
The text was updated successfully, but these errors were encountered:
Hi
We have strict tslint/eslint rules and noticed that the react-async-hook code produces these warnings.
Is it save to include the dependencies that are missing or will that cause unexpected behaviour?
Thanks
/Martin
The text was updated successfully, but these errors were encountered: