Delete node_modules after compile

Question

Delete node_modules after compile

Closed this issue 8 years ago · 3 comments

There usually isn't any reason to include node_modules in your slug, so they should be nuked or it should at least be optional. It looks like compile runs after caching, so I'm going to add it to my compile step for now.

Answer 1 · 2016-08-19T16:29:57.000Z

@aaronjensen doesn't the build directory need all of the app's deps in it? Have you tested this out?

Answer 2 · 2016-08-19T16:41:55.000Z

Yes, see #36. It works well, my slug is 50mb smaller, which makes deployment a little faster and gives more room for assets and such.

Answer 3 · 2016-08-19T16:51:24.000Z

Ahh, I get it now. The node modules are not necessary after compile because the assets have been built. We typically use Node just to build assets in Phoenix. This is definitely useful!