rackerlabs / lambda-uploader

Helps package and upload Python lambda functions to AWS
Apache License 2.0
271 stars 56 forks source link

Fix config defaults to match documentation. #100

Closed brandond closed 8 years ago

brandond commented 8 years ago

The module documentation says:

It is not necessary to set requirements in your config file since the lambda uploader will also check for and use a requirements.txt file.

This is however not actually the default behavior. Actual default behavior is an empty list of requirements, which causes package.py to ignore requirements.txt.

jarosser06 commented 8 years ago

This behavior should have been fixed as of #94 are you sure the default requirements.txt file is still not being picked up in master?

brandond commented 8 years ago

I'll take a look, thanks. Was just running 1.0.0 from PyPI; I'll retest on master.

jarosser06 commented 8 years ago

No problem I'm going to go ahead and close this for now. Thanks

brandond commented 8 years ago

Works for me using current master, thanks!

jarosser06 commented 8 years ago

Excellent. I should be getting a new release out here soon.