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

fix(CI): False doesn't abort build #158

Merged
merged 1 commit into from
Aug 1, 2016

Conversation

Joshua-Anderson
Copy link
Contributor

@Joshua-Anderson Joshua-Anderson commented Aug 1, 2016

I've noticed this behavior earlier with the gcs closure, but it seemed to work otherwise, so I left it elsewhere.

@jchauncey noticed https://ci.deis.io/job/Deis/job/workflow-cli/job/master/69/console, which indicated that e2e tests would restart until the passed, indicating that false was not working.

@deis-bot
Copy link

deis-bot commented Aug 1, 2016

@vdice is a potential reviewer of this pull request based on my analysis of git blame information. Thanks @Joshua-Anderson!

@codecov-io
Copy link

Current coverage is 5.15% (diff: 100%)

Merging #158 into master will not change coverage

@@            master      #158   diff @@
========================================
  Files           44        44          
  Lines         3159      3159          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           163       163          
  Misses        2972      2972          
  Partials        24        24          

Powered by Codecov. Last update 9d52c27...676bc2c

@Joshua-Anderson Joshua-Anderson merged commit 1f412af into deis:master Aug 1, 2016
@Joshua-Anderson Joshua-Anderson deleted the no-retry branch August 1, 2016 23:58
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.

5 participants