dougmoscrop / serverless-plugin-include-dependencies

MIT License
184 stars 40 forks source link

exclude aws-sdk by default #5

Closed linus-amg closed 7 years ago

linus-amg commented 7 years ago

hi, would there be an easy way to exclude the aws-sdk? since it does already exist in the node runtime? maybe a flag or options to force excluding even if its listed as a depedency of a dependency

dougmoscrop commented 7 years ago

ah sorry, this was a feature and I must have forgot to keep it when I refactored; I will add it back (and a regression test ;))

On Tue, Jan 17, 2017 at 4:08 PM, Linus-Alexander Gubenis < notifications@github.com> wrote:

hi, would there be an easy way to exclude the aws-sdk? since it does already exist in the node runtime? maybe a flag or options to force excluding even if its listed as a depedency of a dependency

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dougmoscrop/serverless-plugin-include-dependencies/issues/5, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjzFDB5CgoTfkxzw33NPiH5NZB0UGY-ks5rTS3OgaJpZM4LmKYu .

linus-amg commented 7 years ago

oh yeah, i see: https://github.com/dougmoscrop/serverless-plugin-include-dependencies/blob/v1.0.1/include-dependencies.js#L32