-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Script "make-local" is missing #1184
Comments
Yeah that bit of the FAQ is outdated since we moved to using webpack. I think we should just add a mode where the remaining dependencies are also added to the webpack config and then instead of |
Thanks, Just one more thing: Is it possible to patch the includes to be local? Like, not from |
Google? They should be loaded from It's not possible with the current script sadly, but we should add such a mode again. With browsers increasingly not reusing cached resources between different sites, the utility of a CDN has become very very low. Likely, we should make that the default mode. If you want to have a go, have a look at |
Yes, my mistake, you're right.
I created #1186 for that. But I'm no webpack-expert at all so there might be a thing or two to optimize. |
The FAQ says:
But there is no script
make-local
inpackage.json
The text was updated successfully, but these errors were encountered: