Skip to content

Exit status code 1 after composition errors in service:push#1403

Merged
trevor-scheer merged 2 commits intomasterfrom
trevor/error-composition-failures
Jul 15, 2019
Merged

Exit status code 1 after composition errors in service:push#1403
trevor-scheer merged 2 commits intomasterfrom
trevor/error-composition-failures

Conversation

@trevor-scheer
Copy link
Copy Markdown
Contributor

@trevor-scheer trevor-scheer commented Jul 15, 2019

In line with how we handle errors elsewhere in service:check and service:push, the CLI should exit status code 1 when composition errors are encountered in order to fail CI/CD which uses this command.

TODO:

  • Update CHANGELOG.md* with your change (include reference to issue & this PR)
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass

*Make sure changelog entries note which project(s) has been affected. See older entries for examples on what this looks like.

Copy link
Copy Markdown
Member

@abernix abernix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@trevor-scheer trevor-scheer merged commit a2228a9 into master Jul 15, 2019
@trevor-scheer trevor-scheer deleted the trevor/error-composition-failures branch July 15, 2019 19:21
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