search

kyoshida-aim / RailsTraining

Ruby on Rails研修のPR作成・進捗管理用
https://ky-railstraining-2019.herokuapp.com/
1 stars 0 forks source link

ステップ9-3:CircleCiでRubocopを走らせる #16

Closed kyoshida-aim closed 5 years ago

kyoshida-aim commented 5 years ago

概要

https://github.com/kyoshida-aim/RailsTraining/pull/15 と同じ要領でRubocopを走らせようと思います。

理由

各ステップ毎にコーディング規約を自動的にテストしたいため

確認方法

PR作成時・PRにプッシュした時にPR内で自動実行 image

やっていないこと

エラー詳細をPR側に表示する(CircleCIを見に行かないといけない)

相談事項

一連のPR https://github.com/kyoshida-aim/RailsTraining/pull/15 https://github.com/kyoshida-aim/RailsTraining/pull/13 をMasterブランチに取り込んだら以降のPRは自動テストによるステータスチェックを有効にしようと思います。 image

kyoshida-aim commented 5 years ago

rspec image

rubocop image

エラー発生時にTestSummaryをみることでエラーがわかるようになりました。

vividmuimui commented 5 years ago

https://github.com/kyoshida-aim/RailsTraining/pull/16/commits/4b66c84b1304414bab5ccc4c1aa30d594631317f 修正ってコミット、コミットメッセージを変えるなり、分割したり、どこかのコミットにくっつけるなりしてほしいです。 あと、PRのdescriptionは基本的にはテンプレート( https://github.com/kyoshida-aim/RailsTraining/blob/master/.github/PULL_REQUEST_TEMPLATE.md )に沿って書いてほしいです。 テンプレートの調整が必要だったら調整お願いします。