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

Flank should exit with code 1 when validation issues #642

Merged
merged 1 commit into from
Feb 28, 2020

Conversation

pawelpasterz
Copy link
Contributor

@pawelpasterz pawelpasterz commented Feb 27, 2020

Fixes #448

When validation of yml file fail, process ends with exit code (1).
Prints Valid yml file message otherwise.

@codecov-io
Copy link

Codecov Report

Merging #642 into master will decrease coverage by 0.26%.
The diff coverage is 41.17%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #642      +/-   ##
============================================
- Coverage     77.59%   77.32%   -0.27%     
+ Complexity      614      613       -1     
============================================
  Files            81       82       +1     
  Lines          2321     2329       +8     
  Branches        309      311       +2     
============================================
  Hits           1801     1801              
- Misses          325      334       +9     
+ Partials        195      194       -1

@bootstraponline
Copy link
Contributor

I added Fixes #448 to the pull request body and added a PR template.
55a3039

That will auto close issues once the PRs are merged.

jan-goral
jan-goral previously approved these changes Feb 27, 2020
@pawelpasterz pawelpasterz force-pushed the flank-doctor-exit-code branch from ba181fb to 2a034a1 Compare February 27, 2020 19:43
@pawelpasterz pawelpasterz merged commit 0b63ff2 into master Feb 28, 2020
@pawelpasterz pawelpasterz deleted the flank-doctor-exit-code branch February 28, 2020 07:23
@pawelpasterz pawelpasterz self-assigned this Mar 1, 2020
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.

Flank doctor | Set exit code
4 participants