人間らしくコードレビューするには(パート1) | Yakst

自動化することによりあなたはレビュアーとしてより価値のある貢献ができるようになります。importsの順序やソースコードのファイル名の命名規約などの問題を無視できるならば、機能上の誤りや可読性の問題といった、より関心のある問題にフォーカスすることができます。 オーサーもまた自動化の恩恵を受けます。ケアレスミスを見つけるのに1時間浪費することなく、即座に見つけられます。即座にフィードバックを受けられることで、関係のあることがオーサーの頭に残り、これにより学習が容易となり、修正コストが低くなります。それに加え、彼らが初歩的な誤りについて指摘を受ける必要がある場合、あなたから指摘を受けるよりコンピューターから指摘を受けたほうが彼らの自尊心の観点からはるかに気分がよいわけです。 これらの自動チェックはコードレビューのワークフローの中に入れましょう(例えば、Gitのコミット前のフックやGithubの

[kkamegawa]

youをweに置き換える、とか主語を省くか…もともと日本語だと省かれてしまうから、つい自分のことと思ってしまいがちになってしまうのかもしれない。

[toyama0919]

“コードレビューで「あなた」という言葉を使ってはなりません” なるほどなぁ

[masaru_b_cl]

レビューは複数ラウンド行うことを前提として共有しておくって視点が目からうろこ。意図的に「今は指摘しない」ってやってたけど、相手はその前提しらないから、「何だよ最初から言えよ」って思ってたかもしれない。

[braitom]

相手に嫌な思いをさせないコメントの書き方難しいなあ。そもそも日本語だとあなたとか私たちと言った主語なんて書かないからな。どう書いても自分の責任だと思っちゃうこと多いと思う。

[tbpg]

子を持つと自然と実感するのが端的な誤りの指摘は子のやる気を削ぐ、ということ。意図と影響は異なる。特に含みのない指摘も相手にとっては攻撃として捉えられる可能性があることを意識する。そのあたりに通じる記事

[MnMisato]

“レビューの1ラウンドは最大で1営業日であるべきです。”“レビューを1ヶ月に1度以上辞退せざるをえない場合は、あなたが健全な開発プロセスを維持できるようチームはペースを落とす必要があることを意味します。”

[zou3dazou]

"スタイルなどの細かい指摘はツールにやらせる"

[enmtknt]

“コードレビューを最高優先度にしましょう。時間を取って、コード実際に読んで、フィードバックする際に、レビューをすぐに開始しましょう。理想的には数分以内に、です。”

[kykomi]

だいたい実践してたけど読み直そう

[mizoguche]

“「これは私には理解できません」というのは「これは混乱を招きます」という全ての人間に当てはまるかわからないものであるのに対し、少なくとも客観的な発言です。”

[calpo]

"意見ではなく原則に基づく"

[vanbraam]

"1回のレビューのラウンドで20から50のコメントがつくと危険ゾーン";質問形式にするのはよく使う;サンプル・コードの提示はやりすぎるとreviewerがコードを書く側になってしまうので,余りしない様気を付けてる

[yug1224]

ニンゲン

[kadzuya]

わりと大事な話

[ono_matope]

出来てることあり、出来てないことあり。しかし「高位の事柄を先に指摘して詳細を後回しにする」ってこれ本当に可能なんだろうか。最初からそのCLの本質的な問題を見抜いて適切な方針を提案するの、俺には無理。

[tofu-kun]

人間らしく

[nil-fell]

これに書いてある悪い例全部実践した上に規約にないマイルール押し付けくる人と仕事してたことあるから、今後参考にしたい

[tettekete37564]

命令形にせず相手の許可を得る、もしくは選択肢を委ねる言い方にするというのは世界共通の丁寧語、と昔聞いてからずっと実践してるけど便利よ。

[t-cyrill]

実用的でよい

[masterq]

熟読しよう。。。

[gin0606]

良い

[re_shikajiro]

良いレビューの考察

[yoiIT]

“オーサーが気持ちよくなるための良い方法は、レビューの最中に彼らに贈り物を送るチャンスを見つけることです。開発者はどのような贈り物を喜んで受け取るでしょうか？もちろん、コードのサンプルです。”

[raimon49]

youをweに置き換える、OSSにP-R投げた時も実践してる人を見かけて良い言い回しだなと思ってた

[katsyoshi]

ためになる

[s99e209]

レビューが終わるまで彼らの仕事がブロックされていることがありがちなのでコードレビューは最優先事項とし、数分以内で終わらせるのが理想。

[mitsuhi0909]

すごくよくわかる

[okbm]

いいこと書いてるなー。file change250とか悲しみが生まれるのはほんとつらい

[kikuchi1201]

めちゃ良い記事だ。 コードレビューに関してのモヤモヤが解消できそう。

[o-hayato]

“4. 高位のレベルから始め詳細化する”