Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add include_files option to lambda_package #19

Merged
merged 1 commit into from
Jul 12, 2015

Conversation

dhleong
Copy link
Contributor

@dhleong dhleong commented Jul 9, 2015

This lets you keep credential files, for example,
out of git and npm, but still package them when you deploy

This lets you keep credential files, for example,
out of git and npm, but still package them when you deploy
Tim-B added a commit that referenced this pull request Jul 12, 2015
Add `include_files` option to lambda_package
@Tim-B Tim-B merged commit 9d168b3 into Tim-B:master Jul 12, 2015
@Tim-B
Copy link
Owner

Tim-B commented Jul 12, 2015

Thanks for the pull request!

I've merged it and published a new version.

For anyone else reading this, it's definitely better to keep AWS credentials in ~/.aws/credentials locally and use the function role in production, but I can see this being useful for other types of credentials.

@dhleong
Copy link
Contributor Author

dhleong commented Jul 12, 2015

You're welcome! :) Yes, I'm using it for external API credentials in case I
open source the project.

On Sat, Jul 11, 2015 at 10:19 PM Tim notifications@github.com wrote:

Thanks for the pull request!

I've merged it and published a new version.

For anyone else reading this, it's definitely better to keep AWS
credentials in ~/.aws/credentials locally and use the function role in
production, but I can see this being useful for other types of credentials.


Reply to this email directly or view it on GitHub
#19 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants