Skip to content
This repository has been archived by the owner on Sep 11, 2018. It is now read-only.

vendor.js 3.9Mo size in dist/ #92

Closed
smontlouis opened this issue Sep 17, 2015 · 6 comments
Closed

vendor.js 3.9Mo size in dist/ #92

smontlouis opened this issue Sep 17, 2015 · 6 comments

Comments

@smontlouis
Copy link
Contributor

Is it normal ? Production vendor.js is heavier than developement vendor.js

@MrEfrem
Copy link

MrEfrem commented Sep 17, 2015

Change devtool to 'source-map' in webpack config.
'inline-source-map' can is necessary in the development mode though I don't see a difference since debugging both in ff and in chrome works with 'source-map', but in production precisely there shall be 'source-map'.

@dvdzkwsk
Copy link
Owner

I'll have to check out why the production vendor bundle is larger than development, but the bundle itself will be pretty hefty, since it includes all of these:

[
  'history',
  'immutable',
  'react',
  'react-redux',
  'react-router',
  'redux',
  'redux-devtools',
  'redux-devtools/lib/react'
]

As well as babel-runtime, which I believe isn't unsubstantial. Immutable could be pulled out if you're not using it, and I'd like to remove redux-devtools but can't (to my knowledge) because of a naming conflict with redux-devtools/lib/react and react.

Edit: Thanks for pointing that out @MrEfrem, I should fix that.

@gaearon
Copy link
Contributor

gaearon commented Sep 17, 2015

I'd like to remove redux-devtools but can't (to my knowledge) because of a naming conflict with redux-devtools/lib/react and react.

You can help us by separating LogMonitor into a separate project.
Then there'll be no redux-devtools/lib/react. 😉

@dvdzkwsk
Copy link
Owner

@gaearon haha, now I can't decline. I'll try to get to it sometime next week :).

@smontlouis
Copy link
Contributor Author

@MrEfrem 309ko ! Awesome ! In fact it was inline sourcemap :). Separate sourcemap is 2.7 Mo 👍 . Thank you !

@dvdzkwsk
Copy link
Owner

@bulby97 Saw you post this just as I updated the master branch to use regular source-map. Thanks again to @MrEfrem. Closing this for now then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants