Refactor T.must called on .first
This commit is contained in:
parent
1f55f0c01a
commit
d97933b38c
@ -43,7 +43,7 @@ module Homebrew
|
|||||||
next if !only.empty? && only.exclude?(name)
|
next if !only.empty? && only.exclude?(name)
|
||||||
|
|
||||||
lines.reject! { |line| line.start_with?("#") || line =~ /^\s*$/ }
|
lines.reject! { |line| line.start_with?("#") || line =~ /^\s*$/ }
|
||||||
first_line = T.must(lines.first)
|
first_line = lines.fetch(0)
|
||||||
command = first_line.chomp
|
command = first_line.chomp
|
||||||
command.sub!(/ \$\*$/, "")
|
command.sub!(/ \$\*$/, "")
|
||||||
|
|
||||||
|
@ -61,7 +61,7 @@ module Cask
|
|||||||
tokens.sort.filter_map do |token|
|
tokens.sort.filter_map do |token|
|
||||||
CaskLoader.load_prefer_installed(token, config:, warn: false)
|
CaskLoader.load_prefer_installed(token, config:, warn: false)
|
||||||
rescue TapCaskAmbiguityError => e
|
rescue TapCaskAmbiguityError => e
|
||||||
T.must(e.loaders.first).load(config:)
|
e.loaders.fetch(0).load(config:)
|
||||||
rescue
|
rescue
|
||||||
# Don't blow up because of a single unavailable cask.
|
# Don't blow up because of a single unavailable cask.
|
||||||
nil
|
nil
|
||||||
|
@ -91,7 +91,7 @@ module Homebrew
|
|||||||
raise UsageError, "`brew deps --os=all` is not supported." if args.os == "all"
|
raise UsageError, "`brew deps --os=all` is not supported." if args.os == "all"
|
||||||
raise UsageError, "`brew deps --arch=all` is not supported." if args.arch == "all"
|
raise UsageError, "`brew deps --arch=all` is not supported." if args.arch == "all"
|
||||||
|
|
||||||
os, arch = T.must(args.os_arch_combinations.first)
|
os, arch = args.os_arch_combinations.fetch(0)
|
||||||
eval_all = args.eval_all?
|
eval_all = args.eval_all?
|
||||||
|
|
||||||
Formulary.enable_factory_cache!
|
Formulary.enable_factory_cache!
|
||||||
|
@ -44,7 +44,7 @@ module Homebrew
|
|||||||
if args.no_named?
|
if args.no_named?
|
||||||
git_log(HOMEBREW_REPOSITORY)
|
git_log(HOMEBREW_REPOSITORY)
|
||||||
else
|
else
|
||||||
path = T.must(args.named.to_paths.first)
|
path = args.named.to_paths.fetch(0)
|
||||||
tap = Tap.from_path(path)
|
tap = Tap.from_path(path)
|
||||||
git_log path.dirname, path, tap
|
git_log path.dirname, path, tap
|
||||||
end
|
end
|
||||||
|
@ -464,7 +464,7 @@ module Homebrew
|
|||||||
odie "`brew audit` failed for #{commit[:formula_name]}!"
|
odie "`brew audit` failed for #{commit[:formula_name]}!"
|
||||||
end
|
end
|
||||||
|
|
||||||
new_formula_version = T.must(commits.first)[:new_version]
|
new_formula_version = commits.fetch(0)[:new_version]
|
||||||
|
|
||||||
pr_title = if args.bump_synced.nil?
|
pr_title = if args.bump_synced.nil?
|
||||||
"#{formula.name} #{new_formula_version}"
|
"#{formula.name} #{new_formula_version}"
|
||||||
|
@ -252,7 +252,7 @@ module Language
|
|||||||
raise FormulaUnknownPythonError, self if wanted.empty?
|
raise FormulaUnknownPythonError, self if wanted.empty?
|
||||||
raise FormulaAmbiguousPythonError, self if wanted.size > 1
|
raise FormulaAmbiguousPythonError, self if wanted.size > 1
|
||||||
|
|
||||||
python = T.must(wanted.first)
|
python = wanted.fetch(0)
|
||||||
python = "python3" if python == "python"
|
python = "python3" if python == "python"
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -24,7 +24,7 @@ begin
|
|||||||
|
|
||||||
trap("INT", old_trap)
|
trap("INT", old_trap)
|
||||||
|
|
||||||
formula = T.must(args.named.to_resolved_formulae.first)
|
formula = args.named.to_resolved_formulae.fetch(0)
|
||||||
if args.debug? && !Homebrew::EnvConfig.disable_debrew?
|
if args.debug? && !Homebrew::EnvConfig.disable_debrew?
|
||||||
require "debrew"
|
require "debrew"
|
||||||
formula.extend(Debrew::Formula)
|
formula.extend(Debrew::Formula)
|
||||||
|
@ -36,7 +36,7 @@ begin
|
|||||||
raise "Cannot kill child processes without `pkill`, please install!" unless which("pkill")
|
raise "Cannot kill child processes without `pkill`, please install!" unless which("pkill")
|
||||||
end
|
end
|
||||||
|
|
||||||
formula = T.must(args.named.to_resolved_formulae.first)
|
formula = args.named.to_resolved_formulae.fetch(0)
|
||||||
formula.extend(Homebrew::Assertions)
|
formula.extend(Homebrew::Assertions)
|
||||||
formula.extend(Homebrew::FreePort)
|
formula.extend(Homebrew::FreePort)
|
||||||
if args.debug? && !Homebrew::EnvConfig.disable_debrew?
|
if args.debug? && !Homebrew::EnvConfig.disable_debrew?
|
||||||
|
@ -59,7 +59,7 @@ module UnpackStrategy
|
|||||||
}
|
}
|
||||||
def subextract(extractor, dir, verbose)
|
def subextract(extractor, dir, verbose)
|
||||||
extractor.new(path).extract(to: dir, verbose:)
|
extractor.new(path).extract(to: dir, verbose:)
|
||||||
T.must(dir.children.first)
|
dir.children.fetch(0)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
@ -313,7 +313,7 @@ module Utils
|
|||||||
)
|
)
|
||||||
next if last_thirty_days_match.blank?
|
next if last_thirty_days_match.blank?
|
||||||
|
|
||||||
last_thirty_days_downloads = T.must(last_thirty_days_match.captures.first).tr(",", "")
|
last_thirty_days_downloads = last_thirty_days_match.captures.fetch(0).tr(",", "")
|
||||||
thirty_day_download_count += if (millions_match = last_thirty_days_downloads.match(/(\d+\.\d+)M/).presence)
|
thirty_day_download_count += if (millions_match = last_thirty_days_downloads.match(/(\d+\.\d+)M/).presence)
|
||||||
(millions_match.captures.first.to_f * 1_000_000).to_i
|
(millions_match.captures.first.to_f * 1_000_000).to_i
|
||||||
else
|
else
|
||||||
|
@ -180,7 +180,7 @@ module CPAN
|
|||||||
|
|
||||||
ohai "Updating resource blocks" unless silent
|
ohai "Updating resource blocks" unless silent
|
||||||
Utils::Inreplace.inreplace formula.path do |s|
|
Utils::Inreplace.inreplace formula.path do |s|
|
||||||
if T.must(s.inreplace_string.split(/^ test do\b/, 2).first).scan(inreplace_regex).length > 1
|
if s.inreplace_string.split(/^ test do\b/, 2).fetch(0).scan(inreplace_regex).length > 1
|
||||||
odie "Unable to update resource blocks for \"#{formula.name}\" automatically. Please update them manually."
|
odie "Unable to update resource blocks for \"#{formula.name}\" automatically. Please update them manually."
|
||||||
end
|
end
|
||||||
s.sub! inreplace_regex, resource_section
|
s.sub! inreplace_regex, resource_section
|
||||||
|
@ -109,7 +109,7 @@ module Utils
|
|||||||
raise Interrupt if $CHILD_STATUS.termsig == Signal.list["INT"]
|
raise Interrupt if $CHILD_STATUS.termsig == Signal.list["INT"]
|
||||||
|
|
||||||
if data.present?
|
if data.present?
|
||||||
error_hash = JSON.parse(T.must(data.lines.first))
|
error_hash = JSON.parse(data.lines.fetch(0))
|
||||||
raise rewrite_child_error(error_hash)
|
raise rewrite_child_error(error_hash)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -441,7 +441,7 @@ module PyPI
|
|||||||
|
|
||||||
ohai "Updating resource blocks" unless silent
|
ohai "Updating resource blocks" unless silent
|
||||||
Utils::Inreplace.inreplace formula.path do |s|
|
Utils::Inreplace.inreplace formula.path do |s|
|
||||||
if T.must(s.inreplace_string.split(/^ test do\b/, 2).first).scan(inreplace_regex).length > 1
|
if s.inreplace_string.split(/^ test do\b/, 2).fetch(0).scan(inreplace_regex).length > 1
|
||||||
odie "Unable to update resource blocks for \"#{formula.name}\" automatically. Please update them manually."
|
odie "Unable to update resource blocks for \"#{formula.name}\" automatically. Please update them manually."
|
||||||
end
|
end
|
||||||
s.sub! inreplace_regex, resource_section
|
s.sub! inreplace_regex, resource_section
|
||||||
|
Loading…
x
Reference in New Issue
Block a user