admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Apply suggestions from code review

Browse Source 
Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
Signed-off-by: XuehaiPan <XuehaiPan@pku.edu.cn>
This commit is contained in:
Xuehai Pan 2021-11-24 00:46:02 +08:00 committed by XuehaiPan
parent d1aac8857a
commit 62a63063bb
3 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Library/Homebrew/cmd/update-report.rb
View File

@ -40,14 +40,14 @@ module Homebrew
end
def update_report
return update_report_impl if $stdout.tty?
return output_update_report if $stdout.tty?
redirect_stdout($stderr) do
return update_report_impl
output_update_report
end
end
def update_report_impl
def output_update_report
args = update_report_args.parse
# Run `brew update` (again) if we've got a linuxbrew-core CoreTap

2
Library/Homebrew/style.rb
View File

@ -277,13 +277,11 @@ module Homebrew
end
def shellcheck
# Always use the latest brewed shellcheck
ensure_formula_installed!("shellcheck", latest: true,
reason: "shell style checks").opt_bin/"shellcheck"
end
def shfmt
# Always use the latest brewed shfmt
ensure_formula_installed!("shfmt", latest: true,
reason: "formatting shell scripts")
HOMEBREW_LIBRARY/"Homebrew/utils/shfmt.sh"

2
Library/Homebrew/utils.rb
View File

@ -438,7 +438,7 @@ module Kernel
# Ensure the given formula is installed
# This is useful for installing a utility formula (e.g. `shellcheck` for `brew style`)
def ensure_formula_installed!(formula_or_name, latest: false, reason: "",
def ensure_formula_installed!(formula_or_name, reason: "", latest: false,
output_to_stderr: true, quiet: false)
if output_to_stderr || quiet
file = if quiet