Skip to content

service:check Add ignoreFailures flag#1856

Merged
JakeDawkins merged 3 commits intomasterfrom
adam/20/3/success-on-completion
Mar 31, 2020
Merged

service:check Add ignoreFailures flag#1856
JakeDawkins merged 3 commits intomasterfrom
adam/20/3/success-on-completion

Conversation

@zionts
Copy link
Copy Markdown
Contributor

@zionts zionts commented Mar 27, 2020

When using the GitHub integration, it's often recommended to run
service:check && echo "ignoring output" because GH status will be
updated by the command. Adding this option makes using the command with
GH integration cleaner by exiting with 0 on all schema errors.

Fixes #1852

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.

When using the GitHub integration, it's often recommended to run
service:check && echo "ignoring output" because GH status will be
updated by the command. Adding this option makes using the command with
GH integration cleaner by exiting with 0 on all schema errors.
@zionts zionts requested a review from pcarrier March 27, 2020 23:12
@zionts zionts requested a review from JakeDawkins March 30, 2020 02:45
@JakeDawkins
Copy link
Copy Markdown
Contributor

@zionts I think the naming of this flag could be a little more clear. What do you think about something like --ignoreFailures? I'm just concerned the name of this flag has nothing to do with errors

@JakeDawkins JakeDawkins changed the title service:check Add successOnCompletion flag service:check Add ignoreFailures flag Mar 31, 2020
@JakeDawkins JakeDawkins merged commit 22f20fa into master Mar 31, 2020
@JakeDawkins JakeDawkins deleted the adam/20/3/success-on-completion branch March 31, 2020 16:40
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.

Allow exit 0 on errors reported by schema check

2 participants