diff options
| author | Mike McQuaid | 2017-09-24 20:12:58 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2017-09-24 21:23:59 +0100 |
| commit | cf5fdeef1d8e9fd053121145882835b87eec2a43 (patch) | |
| tree | 1b116e75a955f10925619f11db947f0917a55bf9 /Library/Homebrew/formula.rb | |
| parent | 01e9ec9a9f723b2ef89c7adfaea39f9e4db9aede (diff) | |
| download | brew-cf5fdeef1d8e9fd053121145882835b87eec2a43.tar.bz2 | |
Rubocop: manual rule fixes.
Diffstat (limited to 'Library/Homebrew/formula.rb')
| -rw-r--r-- | Library/Homebrew/formula.rb | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index 69a4cd5aa..d999b9c5f 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -1182,7 +1182,8 @@ class Formula # Returns false if the formula wasn't installed with an alias. def installed_alias_target_changed? target = current_installed_alias_target - target && target.name != name + return false unless target + target.name != name end # Is this formula the target of an alias used to install an old formula? @@ -1446,7 +1447,8 @@ class Formula # True if this formula is provided by external Tap # @private def tap? - tap && !tap.core_tap? + return false unless tap + !tap.core_tap? end # @private @@ -1570,7 +1572,7 @@ class Formula "root_url" => bottle_spec.root_url, } bottle_info["files"] = {} - bottle_spec.collector.keys.each do |os| + bottle_spec.collector.keys.each do |os| # rubocop:disable Performance/HashEachMethods checksum = bottle_spec.collector[os] bottle_info["files"][os] = { "url" => "#{bottle_spec.root_url}/#{Bottle::Filename.create(self, os, bottle_spec.rebuild)}", |