『Code Review Meetup #4 Code Reviewで集まろう』ツイートまとめ
- siderreview
- 1403
- 3
- 0
- 6
SIderわたなべさんから。「快適なコードレビューを目指して」 コードレビューに対する期待。 問題を防ぐ、スタイルを統一する、良い実装、知識の共有。これをめざして真面目にレビューすると大変…。そしてQuerly、Phinderについてのお話へ繋がります。 #codereview4
2018-09-27 20:24:08レビューの流れ。 Diffを見つける、過去にこんなのあったかな?みたいなのを探す。 前にこうだったから心配だよ…と指摘する。この一連をうまく自動化するのに、PHPのPhinderを使ってみる。 例えば、in_arrayの第3引数が省略されてるパターンは要注意なので指摘させたいとか….。 #codereview4
2018-09-27 20:25:34querlyの使い方はlinterと同じイメージだったけど、レビュワーの補助ツールとしてなら誤検知あってもいいな。 #codereview4
2018-09-27 20:26:15大雑把にルールは書いても良いんじゃないか、phinder #codereview4 pic.twitter.com/0CVOPYwndL
2018-09-27 20:26:22過去の障害とからめて、ポストモーテムと一緒に書く。 こういう書き方してしまって障害になっちゃったから、とか。 #codereview4
2018-09-27 20:26:40おっ、 bad pattern も設定ファイルに書けるのか。もしかしてquerly.yml にも?だとするとわざわざソースにbad patternを書いてためすとかしなくて済む? #codereview4
2018-09-27 20:28:03PhinderやQuerlyは、実際に運用すると、誤検知もある。 普通のlinterとの使い方とは少し違う。ローカルで使ったり、CIで回してとりあえずdiffの部分だけをPhinderにかけてチェックするとか。 SIderも近日Phinder対応する予定です! #codereview4
2018-09-27 20:29:14懇親会の様子! ざっくばらんな感じでカジュアルにお話されています🎵😊😄 #codereview4 pic.twitter.com/2Fx6yn9e0O
2018-09-27 20:52:09Phinder YAMLにルールを書き続ければ障害報告ログになるって考え方めっちゃいい。ちゃんと未来に繋がる施策になるから「意識します」や「チェックリストに加えます」よりもずっと生産的。#codereview4
2018-09-27 20:58:49CodeReview Meetup 今回も色々とお話伺えました、ありがとうございました! #codereview4
2018-09-27 22:47:17querlyは以前見つけた時Rubyのみってのと誤検知多いってのでそっ閉じしたが、linterではなくレビュワーへのサジェストツールとして使うのはイケそう。他言語版何とかして作れないかな。 #codereview4
2018-09-27 23:20:40今日はコードレビューミートアップで登壇させていただきました。レビューをやりやすくするために作ったRSpecZ の紹介をしました。 slideshare.net/ssuser7aa1e3/c… #codereview4
2018-09-28 00:21:03RSpecZ の書き方を統一するためにRubocopのカスタムルールを作ろうと思っていて、今考えていることが実現可能かを伊藤さんに話を聞こうと思っていたのに気づいたら懇親会終わって聞き逃してしまた… #codereview4
2018-09-28 00:27:01