Add mjs and jsx filename extensions to file-loader exclude pattern

[iansu]

Closes #3520

See issue for steps to reproduce/test.

[Timer]

@Kingdutch can you confirm this fixes #3520?

[Kingdutch]

@Timer This is the slightly more explicit version of my suggestion in the issue : )

Applied it locally and it fixes the issue 👍

[Timer]

We like explicitness for beginners who eject. 😄

[frenzzy]

@Timer or @gaearon could you release this change please?
By some reason it was not included into v1.0.17, see:

• https://unpkg.com/[email protected]/config/webpack.config.dev.js
• https://unpkg.com/[email protected]/config/webpack.config.prod.js

...and we can't use external libs which file extension is .jsx or .mjs 😢

actual:

loader: require.resolve('file-loader'),
exclude: [/\.js$/, /\.html$/, /\.json$/],

expected:

loader: require.resolve('file-loader'),
exclude: [/\.(js|jsx|mjs)$/, /\.html$/, /\.json$/],

Thanks!

[rtsao]

Even without ejecting, importing a .mjs file via node_modules is broken with without this (which is not in 1.0.17.

[Timer]

It'll be out in the next release.

[gaearon]

This is out in [email protected]! Please let us know if something doesn’t quite work.
https://github.com/facebookincubator/create-react-app/releases/tag/v1.1.0