-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add build/minify step #10
Comments
Do you have preferred tooling?
Or can we go plain npm run-scripts? I can fill a PR if you want. |
I'm pretty apathetic to the specific tooling, as I do not live in the JavaScript packaging universe, I just occasionally glance blows with it. Just keep it simple, y'know? I mean I usually just use a Makefile for this sort of thing. |
Signed-off-by: André Jaenisch <[email protected]>
But you have „Web” in this org :o Just kidding. Find JS part in #11. |
Fixed in #11. Thanks! |
Didn't you hear? Server-side rendering is the future! Google invented it and everything. ;) |
Oh, yeah! Let's rewrite everything in Angular :-D Glad that I could help :-) |
Following on from #9, if there's going to be npm packaging, might as well also add in a build/minification/test step.
The text was updated successfully, but these errors were encountered: