search

kyoshida-aim / RailsTraining

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

ステップ9-2　CircleCiでRSpecを走らせる #15

Closed kyoshida-aim closed 5 years ago

kyoshida-aim commented 5 years ago

概要

CircleCI導入です。

導入にあたって必要なyamlファイルの設定を行なっていきます

理由

各ステップ毎にPR内でテストコードが流れるかどうか確認したかったため

確認方法

PR作成時・PRにプッシュした時に画像のような表示がされる image

やっていないこと

エラー詳細をPRにコメントする(現状CircleCIまで見に行かないといけない)

相談事項

kyoshida-aim commented 5 years ago

image

エラーで失敗した時はエラー内容が表示されるようになりました。

kyoshida-aim commented 5 years ago

成功時 image

Slack通知 image

vividmuimui commented 5 years ago

先週、postgresql周りでCIコケてたと思うんですが、それって結局原因なんだったのでしょうか？(個人的興味で知りたいです)

vividmuimui commented 5 years ago

image

これはマージ必要なコミットでしょうか・・？

kyoshida-aim commented 5 years ago

これはマージ必要なコミットでしょうか・・？

その二つのコミット、一つに混ぜてもいい気はしますが そもそもがVendorディレクトリを削除してもキャッシュの生成・保存ができるかどうかの確認がしたかったので残してもいいと思います。

vividmuimui commented 5 years ago

commitメッセージ調整してもらえば良い気がしますね。テストした結果、必要なかったとわかったんだと思うので、そういうコミットメッセーになれば良いと思います。

- venderを削除してみるテスト
+ vendor/bundleが存在しなくてもbundle時に自動で作られるので削除
+
+ circleciでのbundleの結果のキャッシュが残ってると、circleci上でのvendor/bundleを削除した状態での確認ができないので、
+ bundleの結果のキャッシュも更新する

みたいな！