-
Notifications
You must be signed in to change notification settings - Fork 252
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
[scaffolding-node] Make sure node scaffolding works well with Angular #816
Comments
This is currently failing on
Even though there is definitely a gulpfile.ts in the repo (using https://github.com/mgechev/angular-seed) |
Further investigation: I think the problem is when we run npm install. Currently, we use these options
This returns this error on build
I originally found if I removed the --unsafe-perm option from npm install (therefore running on the npm install things as root) - it got me past the "No gulpfile" found error. However, it then failed later with this error:
This appears to be due to removing the --unsafe-perm arg from $ node install, as also covered here. So...need to find a way to install gulp, but also be able to run the postinstall script. |
EOD update - I got things working up to a point - I needed to manually copy the gulpfile.ts, tools/*, and tsconfig.json to the $CACHE_PATH in order to be able to run $npm install correctly. Now it's failing on some peer dependencies
Haven't figured that out yet, but will pick up again in the morning! |
Update - I did get this working, and wrote up a blog post draft about it here habitat-sh/habitat#4074 However, upon review of the blog post, it was pointed out that it is not common for Angular devs to run the Angular app as a service itself. While this workflow is cool for demo purposes, it would be alienating to those who are used to compiling their Angular apps as static assets and then running them through a web server like nginx. This will likely involve some fundamental rethinking of how the node scaffolding works - but I believe will make it much more powerful. Closing this for now while we rethink that :) |
I think it was @nellshamrell's intention that we close this issue |
Sample project https://github.com/mgechev/angular-seed
The text was updated successfully, but these errors were encountered: