Skip to content

Commit

Permalink
Merge pull request #16002 from samford/update-xcodebuild-type-signature
Browse files Browse the repository at this point in the history
Formula, BuildError: Update type signatures
  • Loading branch information
Bo98 committed Sep 14, 2023
2 parents c953076 + 578c935 commit fe07ced
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Library/Homebrew/exceptions.rb
Original file line number Diff line number Diff line change
Expand Up @@ -479,7 +479,7 @@ class BuildError < RuntimeError
params(
formula: T.nilable(Formula),
cmd: T.any(String, Pathname),
args: T::Array[T.any(String, Pathname, Integer)],
args: T::Array[T.any(String, Integer, Pathname, Symbol)],
env: T::Hash[String, T.untyped],
).void
}
Expand Down
4 changes: 2 additions & 2 deletions Library/Homebrew/formula.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2629,7 +2629,7 @@ def undeclared_runtime_dependencies
#
# # If there is a "make install" available, please use it!
# system "make", "install"</pre>
sig { params(cmd: T.any(String, Pathname), args: T.any(String, Pathname, Integer)).void }
sig { params(cmd: T.any(String, Pathname), args: T.any(String, Integer, Pathname, Symbol)).void }
def system(cmd, *args)
verbose_using_dots = Homebrew::EnvConfig.verbose_using_dots?

Expand Down Expand Up @@ -2794,7 +2794,7 @@ def mkdir(name, &block)
end

# Runs `xcodebuild` without Homebrew's compiler environment variables set.
sig { params(args: T.any(String, Pathname)).void }
sig { params(args: T.any(String, Integer, Pathname, Symbol)).void }
def xcodebuild(*args)
removed = ENV.remove_cc_etc

Expand Down
4 changes: 2 additions & 2 deletions Library/Homebrew/test/exceptions_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -141,11 +141,11 @@ class Baz < Formula; end
end

describe BuildError do
subject { described_class.new(formula, "badprg", %w[arg1 arg2], {}) }
subject { described_class.new(formula, "badprg", ["arg1", 2, Pathname.new("arg3"), :arg4], {}) }

let(:formula) { instance_double(Formula, name: "foo") }

its(:to_s) { is_expected.to eq("Failed executing: badprg arg1 arg2") }
its(:to_s) { is_expected.to eq("Failed executing: badprg arg1 2 arg3 arg4") }
end

describe OperationInProgressError do
Expand Down

0 comments on commit fe07ced

Please sign in to comment.