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

Exit with a non-zero exit code on error #50

Merged
merged 2 commits into from
Jun 6, 2016
Merged

Conversation

lennym
Copy link
Contributor

@lennym lennym commented Jun 6, 2016

Since this script is used in CI environments to deploy lambdas, it should always exit with a non-zero exit code if an error is returned by the AWS SDK so that CI jobs can report as failed.

Fixes #49

Since this script is used in CI environments to deploy lambdas, it should always exit with a non-zero exit code if an error is returned by the AWS SDK so that CI jobs can report as failed.
@nelsonic
Copy link
Member

nelsonic commented Jun 6, 2016

@lennym simple & effective! 👍

@codecov-io
Copy link

codecov-io commented Jun 6, 2016

Current coverage is 100%

Merging #50 into master will not change coverage

@@            master       #50   diff @@
========================================
  Files            7         7          
  Lines          154       154          
  Methods         20        20          
  Messages         0         0          
  Branches        15        15          
========================================
  Hits           154       154          
  Misses           0         0          
  Partials         0         0          

Sunburst

Powered by Codecov. Last updated by 9651747...0f4b69e

@nelsonic nelsonic merged commit a6ef662 into master Jun 6, 2016
@nelsonic nelsonic deleted the bugfix/error-exit-codes branch June 6, 2016 10:44
@nelsonic
Copy link
Member

nelsonic commented Jun 6, 2016

@lennym / @tomgco in case you are interested, the latest version has been published to NPM: dpl@3.4.0
Thanks again for fixing that. 👍

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

Successfully merging this pull request may close these issues.

3 participants