Skip to content

Add Autodll plugin and DllReferencePlugin capability - #131

Closed
moos wants to merge 1 commit into
kitze:masterfrom
moos:moos-autodll
Closed

moos wants to merge 1 commit into
kitze:masterfrom
moos:moos-autodll

Conversation

@moos

@moos moos commented Feb 5, 2018

Copy link
Copy Markdown

This enables DLL generation using the new autodll-webpack-plugin - implementation is similar to this PR react#2710.

I've also added a way to consume DLLs (from autodll or from other repos).

Comment thread package.json
@@ -1,4 +1,5 @@
{
"name": "custom-react-scripts",

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was needed for linklocal to work.

}
if (toArray) {
final.push(result);
Array.isArray(result)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dll reference returns an array - need to merge rather than push.

// https://github.com/jmblog/how-to-optimize-momentjs-with-webpack
// You can remove this if you don't use Moment.js:
new webpack.IgnorePlugin(/^\.\/locale$/, /moment$/),
...customConfig.webpackPlugins,

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dlls are obviously needed for prod builds - so include here. Individual features can disable themselves via the isDev parameter.

@kitze kitze closed this Jun 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants