search

dougmoscrop / serverless-plugin-include-dependencies

MIT License
184 stars 38 forks source link

Is this package already inside serverless? #52

Closed richardtapendium closed 4 years ago

richardtapendium commented 4 years ago

Based on serverless docs here, it looks like this functionality is already in the serverless core.

If so, should we update the README to reflect this so people don't unnecessarily install this plugin?

dougmoscrop commented 4 years ago

No, the serverless core functionality merely excludes dev dependencies and lets you hand-code a list of patterns to include/exclude, rather than excluding all dependencies by default and only adding in the ones that are required() from your handlers or are transitive dependencies of those.

The built-in functionality is also slower, so installing this plugin, plus serverless-plugin-common-excludes, and setting excludeDevDependencies: false results in a faster package step with a smaller output.