Skip to content
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

added material-ui #527

Merged
merged 2 commits into from
Apr 21, 2016
Merged

added material-ui #527

merged 2 commits into from
Apr 21, 2016

Conversation

madvas
Copy link
Contributor

@madvas madvas commented Apr 20, 2016

@Deraen
Copy link
Member

Deraen commented Apr 20, 2016

Hmm, I don't like packaging files from random repositories. Am I correct in assuming that your bundle exists because of mui/material-ui#262?

How is the bundle created? Could we automate the bundle creation here so we could load the source from original repo?

@madvas
Copy link
Contributor Author

madvas commented Apr 20, 2016

Yes, there's a issue that they don't have bundled version and also that it includes react, which is not desired. I create bundle in my other repo, but okay I'll try to automate the process here.

@Deraen Deraen merged commit 016aa4f into cljsjs:master Apr 21, 2016
@Deraen
Copy link
Member

Deraen commented Apr 21, 2016

Looks good now, thanks!

@Deraen Deraen mentioned this pull request Apr 21, 2016
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