変更したファイルにrubocopやjscsを実行して pull requestに自動でコメントする
tl;dr 変更したファイルにrubocopやjscsを実行して、pull requestに自動でコメントさせる方法。 コマンドをパイプでつないで、CIからsaddlerコマンドで書き込みする。 デモリポジトリに rubocop, travis ci, jscs, travis ci エラーになるpull requestしてみてね! saddlerの実行結果イメージ 一番目がjscs, 二番目がrubocop。 流れ diffのあるファイル名を取り出す $ git diff --name-only origin/master README.md bin/run-tests.sh lib/example/travis_ci.rb こんなdiffにrubocop や jscsを実行したい。 diffのあるファイル名を取り出す。 lint実行したいファイルだけに絞り込む $ git diff -
RuboCopのABC Sizeチェックについて調べた - 毒シューマイ
概要 ※この記事はRuboCop v0.28.0が対象です。 https://github.com/bbatsov/rubocop/tree/v0.28.0 Assignment Branch Condition size for method_name is too high. [15.33/15] RuboCopを使っていて、こんな警告が出て困ったので調べた結果をまとめました。 これはABC Sizeというチェック項目で、かいつまむと Assignment : 変数への代入 Branch : メソッド呼び出し Condition : 条件文 上記3項目の合計ポイントをRuboCop君が計算して、基準値を超えると警告が出る、という仕組み。 RuboCopやRuby特有のチェックではなく、Abc Metricといって昔からあるものらしい。 ちなみに、それぞれの頭文字をとってABCとのこと。
Rubocopが良い - rochefort's blog
Atomを使うようになってrubyの静的解析ツールである rubocop を常用するようになりました。もう手放せません。 Atom では yujinakayama/atom-lint というpackageを利用しており、保存時に自動でチェックされていろいろ捗ります。自分ではソースコード書くときは結構注意していたつもりなのですが、いざ使ってみると沢山指摘されます。 rubocopはjslint並みに厳しく指摘してくれるので、copnameというチェック内容単位でoffにする機能があります。（じゃないとエディタがwarningだらけになる） 実はyujinakayama/atom-lint ではこのcopnameが現在は表示されていません。（Add atom-lint.rubocop.showDisplayCopNames #47 by rochefort · Pull Request #82
rubocopによる静的コード解析でRubyのコード品質を保つ | Act as Professional
rubocopはRubyの静的コード解析ツールです。このコード解析を通すことによって、一定のRubyの書き方に統一することができます。また、不要な変数やメソッド名が長すぎるなど、一般的にRubyとして読みやすいコードにするための警告もされます。 こういった警告はRuby coding style and best practicesとしてRuby coding style guideにまとめられおり、Rubyを書くのであれば基本的にはRuby coding style guideを一読しておくことをおすすめします。英語が苦手であれば、翻訳された日本語版も存在します。 なぜ静的コード解析をするのか？静的コード解析し一定の読みやすいコードに統一することによって、人間が誤読する確率を下げることにより、バグなどの混入させる確率を下げる効果があります。また昨今ではGitHubでコードレビューをするよ
実践!! RuboCopプラグイン開発入門 - SideCI TechBlog
こんにちは。RuboCop大好きpockeです! SideCIでは対応ツールの追加など、主にサーバーサイドの開発を担当しています。 今回は、SideCIでも使用しているRubyの静的解析ツール、RuboCop のプラグインの作り方について書こうと思います。 実際にRuboCopプラグインを作りながら手順を解説します! はじめに 今回は、よくあるバグを検出するCopを作ってみましょう。 以下のコードにはバグがありますが、一見しただけだとその存在に気が付かないかも知れません。 def foo raise StandardError "error!" end 本来このコードには、StandardErrorと"error!"の間にカンマが必要です。 カンマが抜けていると「StandardErrorというメソッドを"error!"という引数で呼び出す」という意味になってしまい、想定しない挙動になって
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
