Blogコードレビュー自動化サービス「Sider」を運営するSider株式会社のコーポレートブログです。RuboCopRuboCopなどのLINTツールは、コードに含まれる潜在的な問題を検出しますが、誤検出の問題が常につきまといます。いうプログラムに問題があると言われると、かなり厳しい思いをします。このRuboCopの警告は、Railsアプリケーションの配置されているディレクトリを返すRails.rootからパスを組み立てるときには、式展開などの文字列操作ではなくPathname#joinメソッドを使うことを推奨するものです。"#{user.id}.png")と書いた方が良いという警告です。これはこれで悪くはないのですが、このスクリーンショットに示したプログラムに限って言えば、意味がない警告です。ここでは式展開はログメッセージを組み立てるために使われており、パスはPathname#joinによって構築されています。いう場合には、アビシニアンモードが活躍します。右下の「クローズ」ボタンをクリックして、RuboCopの警告をなかったことにしましょう。無視された警告の情報はSideCIの画面で確認できるようになっています。レビューのときには、SideCIの画面も確認して、無視してはいけない警告が無視されているような場合には、「リオープン」をクリックして元に戻してから開発者に修正を求めましょう。Statusがグリーンに変化します。メッセージには無視された警告の数が表示されます。たくさんの警告が無視されているようなら......ちょっと丁寧にレビューする必要がありそうですね😠SideCIのアビシニアンモードは、LINTツールが検出した問題に対して、開発者がより柔軟な対応を取れるようにするものです。LINTツールの警告が意味のあるものであれば直しますし、誤検出であるとか直す価値がないと自信を持って判断できるものなら無視します。そこまで自信がない場合は、GitHubのコメントで他の開発者・レビューアと話し合います。SideCI開発チームは(開発中から)ずっと利用していますが、クラシックモードには戻れないくらい便利に使っています。アビシニアンモードはリポジトリ設定画面から設定できます。不都合があった場合には、いつでもクラシックモードに戻すことができます。(そのときには、ついでに開発チームまでフィードバックをお送りください🙏)アビシニアンモードでは、SideCIの一部機能、解析ツールが利用できません。現在対応を進めていますので、もうしばらくお待ちください。16:10Tweet関連記事2018-12-03翻訳機能廃止のご案内Siderにて2017年1月から提供してきました「解析結果の翻訳」機...2018-09-20エンジニアが作りたいものを本気で作るための起業という選択肢。追加しましたこんにちは。使用した解析が行えるよ...2015-11-17オフショアラボと日本の品質の考え方の違いと仕組みづくり(コードレビュー編)こんにちは。SideCIの角です。»Siderコードレビュー自動化サービスSiderを提供しています。開発者のインタビュー記事や、活用事例、新しい機能の追加のお知らせ等を更新していきます。RSSはてなブログで読者になるこちらも人気スターをつけました引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。