Skip to content

Move to using webpack for building the library - #144

Merged
claydiffrient merged 2 commits into
masterfrom
development-improvements
Apr 15, 2016
Merged

Move to using webpack for building the library#144
claydiffrient merged 2 commits into
masterfrom
development-improvements

Conversation

@claydiffrient

Copy link
Copy Markdown
Contributor

This allows us to remove the dependency on browserify
completely.

The problem is that the size of the compiled library is 719 kB unminified and 190 kB minified. The current implementation via browserify produces 49 kB unminified and 21 kB minified. That's quite the difference.

Any ideas on how to get that down?

This allows us to remove the dependency on browserify
completely.
@doque

doque commented Apr 13, 2016

Copy link
Copy Markdown
Contributor

Marking react and react-dom as externals in the webpack config significantly reduces build size.

@diasbruno

Copy link
Copy Markdown
Collaborator

@claydiffrient, this PR is great + @doque's PR.
This needs to be merged!! :)

@claydiffrient

Copy link
Copy Markdown
Contributor Author

@diasbruno Looks like there's some conflicts, and I don't have the time right now to resolve them. I'll take a look tonight :)

@diasbruno

Copy link
Copy Markdown
Collaborator

No problem. Just trying to keep this updated.

@diasbruno

Copy link
Copy Markdown
Collaborator

...and thanks to take the time to respond.

@claydiffrient
claydiffrient merged commit 9823bc5 into master Apr 15, 2016
@claydiffrient
claydiffrient deleted the development-improvements branch April 15, 2016 05:36
@claydiffrient

Copy link
Copy Markdown
Contributor Author

@doque @diasbruno now released as v1.1.1

@diasbruno

Copy link
Copy Markdown
Collaborator

awesome, @claydiffrient!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants