This repository has been archived by the owner on Jan 26, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 492
Updating Files Does not Cause HMR Reload #224
Comments
Correction, this has not yet been fixed upstream. I'm not yet sure why I saw the watch being triggered when using that patch - it may relate to webpack/webpack-dev-server#1208, though would not explain sporadic success. |
Ok, can we close this and wait for it to be fixed upstream? |
Agreed, a divergence for this issue would be silly. Please close. |
@wmonk Just an update - I notice many changes just became available upstream today (https://github.com/facebookincubator/create-react-app/blob/master/CHANGELOG.md) |
@mchenryc lol I waited ages and then just decided to update to 17. Thanks for the heads up, better get to work! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I encountered facebook/create-react-app#659 (bug has same title), which has multiple known causes. The cause in my case was not listed, but has already been fixed upstream.
The issue was that my app was nested beneath the folder named*
$monorepo/packages/node_modules
, and ignored by webpack-dev-server's watch, due to an overeager regex forwatchOptions.ignored
.This has already been fixed in CRA and master (1e98d0f, 36cd35d), but is not yet in the released
react-scripts-ts
.This issue should be closed by the next release. Perhaps this will help others.
* I'm using the "Alle" monorepo model, similar to Lerna, adopted by PouchDB
The text was updated successfully, but these errors were encountered: