-
Notifications
You must be signed in to change notification settings - Fork 58
Fixes #15781 - add ssl_cert_file to env check #400
Conversation
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.
I think this is a nice improvement, just a minor coding style nit.
|
||
variables = %w(http_proxy https_proxy ssl_cert_file) | ||
|
||
if variables.map{ |variable| ENV[variable] }.compact.any? |
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.
rubocop wants a space between map
and {
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.
updated
|
||
if variables.map{ |variable| ENV[variable] }.compact.any? | ||
$stderr.puts "Please unset the following environment variables before running the installer: #{variables.join(', ')}" | ||
exit 1 |
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.
And trailing whitespace
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.
updated!
Thanks for the review @ekohl ! |
(cherry picked from commit 9b9daf3)
(cherry picked from commit 9b9daf3)
Adds hammer-foreman repo.
No description provided.