http://post.simplie.jp/posts/34

[aukusoe]

「キャッシュが必要な箇所はないか」 激しく誤読してしまった。いやうん、あれだ、レビューする環境を作るのにも予算はいるから一概に誤読ではないな。

[kastro-iyan]

参考になるけど、「もっといいコードが書けないか」とかは抽象的すぎて逆効果だと思う。具体的に書かないと誰もやらないんだよね

[verda]

チェックリストより前に規約がいりそう 「変数名は適切か」みたいなのの適切って何を持って判断するんだろう？

[t-murachi]

「もっとよいコードにできないか」<こういう曖昧な項目がないかを、チェックリスト作成時にしっかりとレビューするべき。

[ebo-c]

うちはSHOULDがあるんだけど、MUSTかIMOかの2値な方がよいのかも。nits - nitpick 細かいことにうるさく言う、あら探しをする

[ledsun]

「まずチェックリストを作る」が大事だよね。（チェックリストの改善ができるので）

[greencoffeemaker]

指摘分類が重要度なのが良いな

[ryu39]

リファクタリングは、修正対象なら全然いいんですが、修正範囲外もついでに直した的なものは別プルリクにしてほしい自分がいます。

[mmorita44]

チェックリストの大概の項目は極力Checkstyle等で自動化したい。

[mtmt101jp]

これ参考にしてつくろっと

[miho-satoh-satomi]

コードレビュー チェック 項目 メモ

[kimihito]

“レビューのコンテキストを明確にする WIP段階：ビジネスロジックの視点（全体像）で指摘する マージ前：コードの質（細部）を含めて指摘する”

[UDONCHAN]

参考になる

[dorayakikun]

コードレビューの指針について

[efcl]

コードレビューのチェックリスト

[kukita]

【#プログラミング】「コードレビューにおけるチェックリストとそのやり方」自分用メモφ(..) #コードレビュー #レビュー →

[kiri2mau]

意識する

[sivadselim]

May 26, 2016 at 0557PM - コードレビューにおけるチェックリストとそのやり方 | SimpliePost

[qaz76]

ウチは敢えてチケット毎のエクセルにしてます。セルフチェック&署名→マージリクエスト→受け入れレビュー&署名→マージというgit-flowアレンジ。

[kaputte]

有用

[highfrontier]

study, test コードレビューにおけるチェックリストとそのやり方

[nashirox]

いいチェックリスト用意できる人って尊敬する

[karikari1255]

むしろコード規約なんて作りこみたくないからレビュー文化を作るのがいいんですよ。そのチームにとっての「適切な○○」は実際のコード上での議論から自然と導かれて集合知になっていく

[kamayan1980]

コメント書く派、コード読めば分かる派、トレンド乗っかり派、古典派、関数名を英文にするなハゲ派、ハゲてない派、髪の話はやめて派など様々な宗派の方たちがファイトクラブして出来上がるのが良いチェックリスト。

[naqtn]

この例は“出来るチーム”がより楽にやるためのものであって、“出来ないチーム”がそれなりに出来るようになるための処方箋ではない気がする。

[kaipu1224]

なんでもかんでも文字列連結をStringBuilder使えって言われた嫌な思いでしか無い

[w1234567]

あんま好きになれないルール、主観的で粒度バラバラだし特にデバッグ用のコードいれるなとか大きなお世話だわ、どうせテメーはデバッグも運用もしないくせに黙っとけよと思う

[masayoshinym]

“もっとよいコードにできないか”やっぱここ引っ掛かるよね。それ以外は参考になりそう。

[yoshikidz]

チェックリストも大事だけれど、レビュアーと開発者の意識大事

[s99e209]

コードレビューのチェックリストが参考になる。

[koogawa]

確かにMUSTなのかSHOULDなのか明示するのは重要かも

[inoji9]

最初から言っておいてくれよ を避けるのに有効。

[kgrock]

コードレビューにおけるチェックリストとそのやり方

[purosuta]

参考になります