From e7052fad0d970f1641dbd11a892fe9676832d08f Mon Sep 17 00:00:00 2001 From: Colin Dean Date: Thu, 21 Jul 2022 13:19:54 -0400 Subject: [PATCH] Use heredoc for multiline error and store untapped official taps check Review feedback on https://github.com/Homebrew/brew/pull/13581#pullrequestreview-1046232447 --- Library/Homebrew/brew.rb | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/Library/Homebrew/brew.rb b/Library/Homebrew/brew.rb index cfb09f5353..7f52b5a483 100644 --- a/Library/Homebrew/brew.rb +++ b/Library/Homebrew/brew.rb @@ -103,13 +103,14 @@ begin possible_tap = OFFICIAL_CMD_TAPS.find { |_, cmds| cmds.include?(cmd) } possible_tap = Tap.fetch(possible_tap.first) if possible_tap - if !possible_tap || possible_tap.installed? || Tap.untapped_official_taps.include?(possible_tap.name) - blocked_tap = possible_tap && Tap.untapped_official_taps.include?(possible_tap.name) + if !possible_tap || + possible_tap.installed? || + (blocked_tap = Tap.untapped_official_taps.include?(possible_tap.name)) if blocked_tap - [ - "`brew #{cmd}` is unavailable because #{possible_tap.name} was manually untapped.", - "Run `brew tap #{possible_tap.name}` to reenable `brew #{cmd}`.", - ].each { |ln| onoe ln } + onoe <<~EOS + `brew #{cmd}` is unavailable because #{possible_tap.name} was manually untapped. + Run `brew tap #{possible_tap.name}` to reenable `brew #{cmd}`. + EOS end # Check for cask explicitly because it's very common in old guides odie "`brew cask` is no longer a `brew` command. Use `brew --cask` instead." if cmd == "cask"