You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We disable Performance/RedundantBlockCall because "[it] conflicts with Sorbet since we need to capture the block to type it."
There is precedent for RuboCop extensions patching cops defined somewhere else, and this seems like a reasonable thing to patch. If the method definition is prefixed with a sig, this cop should not complain about capturing the block argument, even if yield is used.
The text was updated successfully, but these errors were encountered:
We disable
Performance/RedundantBlockCall
because "[it] conflicts with Sorbet since we need to capture the block to type it."There is precedent for RuboCop extensions patching cops defined somewhere else, and this seems like a reasonable thing to patch. If the method definition is prefixed with a
sig
, this cop should not complain about capturing the block argument, even ifyield
is used.The text was updated successfully, but these errors were encountered: