Lint/RedundantWithIndex copのご提案

鋭意 PR レビュー中の cop であり一週間くらい後手に回っていたもの。それにようやく手を入れたりしていた。もともと JRuby のバグを発端に作ろうと思った cop であるが、RuboCop 本体にも offense に掛かる箇所があることから、世の中には不要な with_index を使ったままのコードがあるのかもしれないということで作った。

github.com

フィードバックを読んでいたら、もしかしたらブロック引数の数に依存した作りでは問題があって再設計が必要かもと考えていたのが後手に回った要因だったが、いろいろと試したところブロック引数の数に依存することで問題なさそうだったので、ほとんど現状維持とした。そして、この PR での現状のやりとりで一番変わっているのは cop の名前だったりする。