Third try at fixing build
This commit is contained in:
parent
2e2e09ed2b
commit
c197c2f4fd
1 changed files with 11 additions and 11 deletions
|
@ -110,21 +110,21 @@ module.exports = (env, argv) => {
|
||||||
{
|
{
|
||||||
test: /\.(ts|js)x?$/,
|
test: /\.(ts|js)x?$/,
|
||||||
include: (f) => {
|
include: (f) => {
|
||||||
|
// our own source needs babel-ing
|
||||||
|
if (f.startsWith(path.resolve(__dirname, 'src'))) return true;
|
||||||
|
|
||||||
// we use the original source files of react-sdk and js-sdk, so we need to
|
// we use the original source files of react-sdk and js-sdk, so we need to
|
||||||
// run them through babel.
|
// run them through babel. Because the path tested is the resolved, absolute
|
||||||
if (f.startsWith(path.resolve(__dirname, 'node_modules', 'matrix-js-sdk', 'src'))) return true;
|
// path, these could be anywhere thanks to yarn link. We must also not
|
||||||
if (f.startsWith(path.resolve(__dirname, 'node_modules', 'matrix-react-sdk', 'src'))) return true;
|
// include node modules inside these modules, so we add 'src'.
|
||||||
|
if (f.includes(path.join('matrix-js-sdk', 'src'))) return true;
|
||||||
|
if (f.includes(path.join('matrix-react-sdk', 'src'))) return true;
|
||||||
|
|
||||||
// but we can't run all of our dependencies through babel (many of them still
|
// but we can't run all of our dependencies through babel (many of them still
|
||||||
// use module.exports which breaks if babel injects an 'include' for its
|
// use module.exports which breaks if babel injects an 'include' for its
|
||||||
// polyfills: probably fixable but babeling all our dependencies is probably
|
// polyfills: probably fixable but babeling all our dependencies is probably
|
||||||
// not necessary anyway).
|
// not necessary anyway). So, for anything else, don't babel.
|
||||||
if (f.startsWith(path.resolve(__dirname, 'node_modules'))) return false;
|
return false;
|
||||||
// anything else gets babeled (our own source files, and also modules that
|
|
||||||
// are yarn linked from somewhere else because this tests the absolute,
|
|
||||||
// resolved path, so react-sdk and js-sdk fall under this case in a standard
|
|
||||||
// dev setup. This will presumably start running any other module through
|
|
||||||
// babel if yarn linked... caveat emptor.
|
|
||||||
return true;
|
|
||||||
},
|
},
|
||||||
loader: 'babel-loader',
|
loader: 'babel-loader',
|
||||||
options: {
|
options: {
|
||||||
|
|
Loading…
Reference in a new issue