shen/brew
1
0
Fork 0
mirror of https://github.com/Homebrew/brew.git synced 2024-11-25 16:33:34 +08:00
Code Issues Actions 11 Packages Projects Releases Wiki Activity

Remove :final from Requirement#satisfied? sig

Browse Source
This commit is contained in:
Douglas Eichelberger 2024-11-24 10:18:45 -08:00
parent 3b1d59d8c4
commit ccfc8bfbe6
2 changed files with 6 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Library/Homebrew/requirement.rb
View File

@ -60,8 +60,9 @@ class Requirement
s
end
# Pass a block or boolean to the satisfy DSL method instead of overriding.
sig(:final) {
# Overriding {#satisfied?} is unsupported.
# Pass a block or boolean to the satisfy DSL method instead.
sig {
params(
env: T.nilable(String),
cc: T.nilable(String),
@ -82,8 +83,9 @@ class Requirement
true
end
# Pass a boolean to the fatal DSL method instead of overriding.
sig(:final) { returns(T::Boolean) }
# Overriding {#fatal?} is unsupported.
# Pass a boolean to the fatal DSL method instead.
sig { returns(T::Boolean) }
def fatal?
self.class.fatal || false
end

11
Library/Homebrew/test/requirement_spec.rb
View File

@ -59,17 +59,6 @@ RSpec.describe Requirement do
describe "#fatal is omitted" do
it { is_expected.not_to be_fatal }
end
describe "in subclasses" do
it "raises an error when instantiated" do
expect do
Class.new(described_class) do
def fatal? = false
end
end
.to raise_error(RuntimeError, /\AThe method `fatal\?` on #{described_class.name} was declared as final/)
end
end
end
describe "#satisfied?" do