Merge pull request #16539 from dduugg/fixed-fixmes
Remove some fixed FIXMEs
This commit is contained in:
commit
8392f70620
@ -471,8 +471,7 @@ class Pathname
|
||||
else
|
||||
# Length of the longest regex (currently Tar).
|
||||
max_magic_number_length = 262
|
||||
# FIXME: The `T.let` is a workaround until we have https://github.com/sorbet/sorbet/pull/6865
|
||||
T.let(binread(max_magic_number_length), T.nilable(String)) || ""
|
||||
binread(max_magic_number_length) || ""
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
@ -14,13 +14,6 @@ module RuboCop
|
||||
sig { overridable.params(cask_stanza_block: RuboCop::Cask::AST::StanzaBlock).void }
|
||||
def on_cask_stanza_block(cask_stanza_block); end
|
||||
|
||||
# FIXME: Workaround until https://github.com/rubocop/rubocop/pull/11858 is released.
|
||||
def find_end_line(node)
|
||||
return node.loc.end.line if node.block_type? || node.numblock_type?
|
||||
|
||||
super
|
||||
end
|
||||
|
||||
sig { params(block_node: RuboCop::AST::BlockNode).void }
|
||||
def on_block(block_node)
|
||||
super if defined? super
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user