From 3dae2d88c74f22afecfc777764ddc465d129fa4c Mon Sep 17 00:00:00 2001 From: Joe Lim Date: Mon, 2 Oct 2017 19:57:02 -0700 Subject: [PATCH] Watch for changes in `src/**/node_modules` (#3230) * Allow the dev server to watch for changes in src/node_modules * fix eslint error * fix broken regex * handle trailing slash edge case for file paths Closes #2760 Fixes #3223 --- config/webpackDevServer.config.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/config/webpackDevServer.config.js b/config/webpackDevServer.config.js index 25b42d0de8b..a48a1fbc3a6 100644 --- a/config/webpackDevServer.config.js +++ b/config/webpackDevServer.config.js @@ -10,6 +10,7 @@ const errorOverlayMiddleware = require('react-dev-utils/errorOverlayMiddleware'); const noopServiceWorkerMiddleware = require('react-dev-utils/noopServiceWorkerMiddleware'); +const path = require('path'); const config = require('./webpack.config.dev'); const paths = require('./paths'); @@ -72,8 +73,13 @@ module.exports = function(proxy, allowedHost) { quiet: true, // Reportedly, this avoids CPU overload on some systems. // https://github.com/facebookincubator/create-react-app/issues/293 + // src/node_modules is not ignored to support absolute imports + // https://github.com/facebookincubator/create-react-app/issues/1065 watchOptions: { - ignored: /node_modules/, + ignored: new RegExp( + `^(?!${path.normalize(paths.appSrc + '/')}).+[\\/]node_modules[\\/]`, + 'g' + ), }, // Enable HTTPS if the HTTPS environment variable is set to 'true' https: protocol === 'https',