Kent C. Dodds
Kent C. Dodds
Fantastic! Thanks for the pull request! Let me add a few inline comments for you. I have some feedback :-)
Looking pretty good. When you've completed what I've noted, could you follow the instructions on how to [squash your commits](https://egghead.io/lessons/javascript-how-to-squash-multiple-git-commits?series=how-to-contribute-to-an-open-source-project-on-github)? When you do, you'll get a chance to update your...
ping :)
It's quite possible that your local `master` branch is behind this project's `master`. You'll want to set this repo as an `upstream` and have your local `master` track `upstream/master` as...
Nice catch! I'm pretty sure that this change is due to a recent change in React v17: https://github.com/facebook/react/pull/17925 I'll need to re-record this video. Thanks for letting me know :)...
Because the App component is never removed from the page.
I'm afraid I have no more to offer than what is suggested in that issue. Did you try installing watchman?
`kcd-scripts` has a few nice things built-in, but it's unlikely to cause this problem. However, it's possible. You should be able to run `npx jest` and it'll run all the...
Odd. Unfortunately I don't have the bandwidth to look into this.
Conversation locked to avoid conversation about any of these points. These are more notes than anything else. If you want to discuss any of these, open another issue.