-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Let cask doctor exit with 1 #3631
Conversation
- Remove :failed - Use Checkable to store the status and warnings/errors - Refactor the methods using audit as basis
/ping @reitermarkus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small changes, otherwise 👍.
check_environment_variables | ||
|
||
puts summary unless success? | ||
raise CaskError, "Your system is not ready for Cask." unless success? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe “There are some problems with your setup.”.
end | ||
|
||
def summary_header | ||
"Cask's Doctor checkup" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capitalize “checkup”.
@reitermarkus, can you check if this is all right? |
Thanks @amyspark! |
Thanks, @amyspark! I was about to get back to this today, but looks like Mike beat me by a day. 😉 |
brew tests
with your changes locally?(I am reopening #3418 with this pull request, since it got stale)
This pull request fixes Homebrew/homebrew-cask#40198 by raising a CaskError if one or more of the checks fails.
Please review!