Skip to content

Conversation

@tannerlinsley
Copy link
Contributor

Possible usages are:

src/Chart.js (umd)
dist/Chart.js (vanilla)
dist/Chart.min.js (vanilla minified)
dist/Chart.bundle.js (vanilla with Moment.js)
dist/Chart.bundle.min.js (vanilla with Moment.js minified)

@etimberg
Copy link
Member

😮

nice! Not sure what the test failures are, but this would be great. @davis you might be interested in this

Possible usages are:

src/Chart.js (umd)
dist/Chart.js (ChartJS)
dist/Chart.min.js (ChartJS minified)
dist/Chart.bundle.js (ChartJS with Moment.js)
dist/Chart.bundle.min.js (ChartJS with Moment.js minified)
@tannerlinsley
Copy link
Contributor Author

Closes #1898

@tannerlinsley
Copy link
Contributor Author

Closes #1873

@tannerlinsley
Copy link
Contributor Author

Closes #1479

This was referenced Feb 12, 2016
@timcroydon
Copy link

Looks like this branch addresses #1989 too.

@etimberg
Copy link
Member

Might also address #1847

@etimberg
Copy link
Member

Only thing I'd suggest adding to this is some samples & docs. Maybe show a sample where we pull it in with requireJS. It would also be great to have something in the docs about using it with require / browserify / webpack since it's a fairly common question that we get.

@davis
Copy link
Contributor

davis commented Feb 13, 2016

nice!

@etimberg
Copy link
Member

@tannerlinsley doc updates look good. you can go ahead and merge any-time then close out the above issues 😄

@etimberg etimberg added this to the Version 2.0 milestone Feb 13, 2016
tannerlinsley added a commit that referenced this pull request Feb 13, 2016
@tannerlinsley tannerlinsley merged commit d9af94a into v2.0-dev Feb 13, 2016
@etimberg etimberg deleted the browserify-build-system branch February 15, 2016 15:58
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.

5 participants