-
Notifications
You must be signed in to change notification settings - Fork 989
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
Why does the prestart:prod script remove the dist folder? #65
Comments
|
OK, thanks for the precision. It looks awkward to build the application on each startup, especially in a container environment. I usually build my application only once (on delivery) then start it from the |
I agree with @fvilers - this should be a |
+1 I agree. The start of your application should assume it has already been built via your build system. For containerized environments, this is especially something that should be |
We actually changed that already. |
Hi! I'm trying to deploy my first NestJS application and I may be doing something wrong. I can't find the documentation about building for production on the NestJS website.
From the available npm scripts, I've built the application using the
build
script and I'm trying to run it using thestart:prod
script. What bugs me is why do we have aprestart:prod
script that removes the whole dist folder? Thestart:prod
script can't start anything without it.I suppose this is a bug and that the
prestart:prod
script is indeed aprebuild
script. I'll gladly submit a PR if someone from the team confirms that bug.The text was updated successfully, but these errors were encountered: