numo-labs / aws-lambda-helper

:lollipop: Collection of helper methods for lambda
GNU General Public License v3.0
22 stars 2 forks source link

Move aws-sdk to full dependency #49

Closed lennym closed 8 years ago

lennym commented 8 years ago

The module is require-d at runtime so should be listed as a full dependency.

nelsonic commented 8 years ago

@lennym we have deliberately excluded aws-sdk from aws-lambda-helper because it's available in Lambda containers see: http://docs.aws.amazon.com/lambda/latest/dg/current-supported-versions.html

including aws-sdk as a dependency here will add 2mb to every lambda and wait time for every deployment because the "production" node_modules need to be dowloaded and zipped each time... a simpler approach is to include aws-sdk as a devDependency in the package.json of the Lambda function you are building and leave it out of aws-lambda-helper.