You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
On the Serverless FAQ page there is mention of using webpack and bundling node_modules to help with bootstrap time.
However, instead of using the default webpack configuration that Nest CLI ships with, we'll make sure to bundle all dependencies (node_modules) together, as follows
Expected behavior
Have a hint or warning that it's not recommended to bundle node_modules with Node.js applications, and that the example is for demonstration purposes only.
What is the motivation / use case for changing the behavior?
While trying to implement this to improve init durations on Lambda, I came across some of the issues mentioned in this ticket and a comment which recommends against bundling Node.js applications with node_modules. The motivation for this change is to prevent others from spending time trying to implement this while they'll likely run into the same issue. I would be happy to try submitting a PR for this if the maintainer(s) agree with the change.
The text was updated successfully, but these errors were encountered:
JustinKeller
added a commit
to JustinKeller/docs.nestjs.com
that referenced
this issue
Jul 27, 2023
I'm submitting a...
Current behavior
On the Serverless FAQ page there is mention of using webpack and bundling node_modules to help with bootstrap time.
Expected behavior
Have a hint or warning that it's not recommended to bundle node_modules with Node.js applications, and that the example is for demonstration purposes only.
What is the motivation / use case for changing the behavior?
While trying to implement this to improve init durations on Lambda, I came across some of the issues mentioned in this ticket and a comment which recommends against bundling Node.js applications with node_modules. The motivation for this change is to prevent others from spending time trying to implement this while they'll likely run into the same issue. I would be happy to try submitting a PR for this if the maintainer(s) agree with the change.
The text was updated successfully, but these errors were encountered: