jlfwong / speedscope / 737

47%

push

travis-ci

Fix bug with bad caching of action creators (#281)

Profile switching was subtly broken because action creators weren't being correctly re-bound due to a missing dependency in a `useCallback` call.

I also tried to reduce boilerplate in this PR by adding additional exhaustive deps protection via eslint for `useSelector`, `useAppSelector`, and `useActionCreator`. The removes the need for using `useCallback` or each of those.

Fixes #280

875 of 1995 branches covered (43.86%)

Branch coverage included in aggregate %.

29 of 29 new or added lines in 7 files covered. (100.0%)

2236 of 4627 relevant lines covered (48.33%)

26642.0 hits per line