-
Notifications
You must be signed in to change notification settings - Fork 334
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
Build for the consumer #345
Comments
@lastmjs Are you talking about the browser build? I'd accept a PR to include the browser build in the npm package. But I'm strongly against keeping built files in the git repo. |
Great, I might take you up on that then. |
It looks like |
@gabejohnson I'll make a release soon. |
I'd love to have a browser build easily available. I'm trying to work out how to build it from the npm package but I'm a beginner with node. Makes it hard to get at. |
I'm interested in this as well. I'm a bit confused about the state of the releases anyway. It seems that the release tags starting at 1.7.0 no longer contain the built browser files, which seems to also affect the bower releases too. I tried toying a bit with the Were the latest npm releases done without the release script by any chance ? |
I think I figured it out. See #368 as a proposed change for this. |
I'm running into issues with this repo because the assets aren't built already for me. Now I'm going to have to manually go in and do an npm install and run the script to build. I might have to fork the repo so that I can change the package.json to do this for me, or just commit the built files. Why not build it for us? Or just add a post install script or something?
The text was updated successfully, but these errors were encountered: