UnpackStrategy: Make #dependencies non-nilable

`T.nilable` was removed from the `P7Zip#dependencies` type signature
in a previous commit, as `UnpackStrategy#dependencies` doesn't allow
for a nilable return type. This updates the type signature for the
other `#dependencies` methods to also remove `T.nilable`, as they
would produce a runtime error otherwise.
This commit is contained in:
Sam Ford 2024-07-28 10:12:18 -04:00
parent 0a658a3290
commit 6918160fd9
No known key found for this signature in database
GPG Key ID: 7AF5CBEE1DD6F76D
9 changed files with 9 additions and 9 deletions

View File

@ -17,7 +17,7 @@ module UnpackStrategy
path.magic_number.match?(/.{59}#{Regexp.escape(mime_type)}/) path.magic_number.match?(/.{59}#{Regexp.escape(mime_type)}/)
end end
sig { returns(T.nilable(T::Array[Cask::Cask])) } sig { returns(T::Array[Cask::Cask]) }
def dependencies def dependencies
@dependencies ||= T.let([Cask::CaskLoader.load("adobe-air")], T.nilable(T::Array[Cask::Cask])) @dependencies ||= T.let([Cask::CaskLoader.load("adobe-air")], T.nilable(T::Array[Cask::Cask]))
end end

View File

@ -24,7 +24,7 @@ module UnpackStrategy
verbose: verbose:
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["cabextract"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["cabextract"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
false false
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["unar"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["unar"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\A..-(lh0|lh1|lz4|lz5|lzs|lh\\40|lhd|lh2|lh3|lh4|lh5)-/n) path.magic_number.match?(/\A..-(lh0|lh1|lz4|lz5|lzs|lh\\40|lhd|lh2|lh3|lh4|lh5)-/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["lha"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["lha"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\ALZIP/n) path.magic_number.match?(/\ALZIP/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["lzip"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["lzip"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\A\]\000\000\200\000/n) path.magic_number.match?(/\A\]\000\000\200\000/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["xz"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["xz"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\ARar!/n) path.magic_number.match?(/\ARar!/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["libarchive"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["libarchive"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\A\xFD7zXZ\x00/n) path.magic_number.match?(/\A\xFD7zXZ\x00/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["xz"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["xz"]], T.nilable(T::Array[Formula]))
end end

View File

@ -16,7 +16,7 @@ module UnpackStrategy
path.magic_number.match?(/\x28\xB5\x2F\xFD/n) path.magic_number.match?(/\x28\xB5\x2F\xFD/n)
end end
sig { returns(T.nilable(T::Array[Formula])) } sig { returns(T::Array[Formula]) }
def dependencies def dependencies
@dependencies ||= T.let([Formula["zstd"]], T.nilable(T::Array[Formula])) @dependencies ||= T.let([Formula["zstd"]], T.nilable(T::Array[Formula]))
end end