Jump to content

健全なコード レビュー文化のための指針

From mediawiki.org
This page is a translated version of the page Guidelines for a healthy code review culture and the translation is 69% complete.

コード レビューは感情の労働です。私たちは、自分が書いたコードに至るまで、この仕事に感情移入している存在なのです。 フィードバックを与えたり受け取ったりするのは大変なことで、健全なコード レビュー文化を構築するためには、この負荷を全員で共有する必要があります。

コード レビューは、表面的には、物事が壊れるのを防ぎ、コードの健全性を維持するためのものです。 実際には、それ以上のもの (のはず) です。

コードレビューの目標

  1. 不具合や脆弱性が本番環境のコードに反映されるのを防ぐために協力し合うこと
  2. 保守性を促進し、将来の不満や混乱を防止すること
  3. 協力者が学び、成長するための教育の機会を提供し、新しい協力者を呼び込むこと
  4. 協調作業を通じてコードの共有理解、オーナーシップ、説明責任を育み、より機能的で充実した貢献者チームを実現すること

最終的に、コード レビューは対話であり、私たちの仕事のほとんどは遠隔地や非同期で行われるため、この運動にとって特に重要なものです。 私たちは、コード レビュアーのコミュニティの一員であり、このコミュニティ内で信頼関係を構築することが、コードレビューの目標を達成し、既存のコミュニティ メンバーをサポートし、新しいメンバーを導入するために役立つと考えています。

健全なコード レビュー文化とは?

健全なコードレビュー文化とは、恐れることなくフィードバックを歓迎するものです。
—Cindy Cicalese 氏、ウィキメディア財団の主任ソフトウェア エンジニア

これを詳しく見てみましょう:

  • フィードバックは歓迎される: 健全なコードレビュー文化では、フィードバックを受け取ることは、本番環境に進む前に問題点が発見され、知識が共有され、人々が協力してコードを改善することを意味します。このような文化で作業することには、やりがいがあります。
  • 恐れなく: 心理的安全性は、健全なコードレビュー文化の基盤です。[1] 人々はフィードバックの提供や受け取りに快適さを感じる必要があります。 パッチを提出したり、批評を行うことで圧迫感を与えることがあり、このプロセスには信頼が必要です。 安心感を持つ人々は新しいアイデアを提案し、実験し、成長できます。 安心感を感じない人々は最終的に貢献を止めて去っていってしまいます。

私たちは、自分たちの価値観を明確にし、それを作業の進め方に適用することで、このような文化を作り出せます。

健全なコードレビュー文化の価値観

エゴよりも尊重と共感

  • 私たちのコードが私たちを定義しないこと: 私たちは書いたコードに深く関わっていますが、コーディングは私たちがやることであり、誰であるかではありません。コーダーではなくコードを批評しましょう。
  • 思いやりをもって先導する: まず自分から始め、それを誰にでも広げましょう。私たちは皆今できる最善を尽くしています。[1]
  • 信頼を築く: 優しさ、共感、好奇心が共同作業者間の関係を築けます。信頼は心理的安全性につながり、素晴らしい作業と幸せな貢献者につながります。
  • 能力を想定する: 無能力を想定するのではなく、質問しましょう。何かを誤解しているかもしれないのはあなたかもしれません。[2]
  • 権力構造に気を配る: あなたよりも経験が少ない人の意見に耳を傾けましょう。声の小さい人たちを支援しましょう。他の人に譲りましょう。


競争よりも協力

  • 協力に重点を置くこと: 協力に注力すると、エゴが邪魔をすることはありません。協力は、多くの人々が貢献することを可能にし、より良い製品を生み出します。
  • 協力の課題を認識する: 私たちの中には、トーンを管理するのが難しい人、好きではない解決策に譲ることに苦労する人、批判的であることを避ける人もいます。みんな違うけど、成長できるんだということを覚えておいてください。
  • フィードバックを贈り物として受け入れる: 健全な文化では、すべてのフィードバックがコードを改善した、何かを教えてくれた、あるいは考えさせてくれたものとして歓迎されます。
  • 好奇心と実験を促進する: 安全な環境で、私たちは遊びを通じて学び、革新し、より楽しく取り組むことができます。
  • 謙虚に異論を述べる: 異論がある場合は、尊重をもって自分の意見を述べ、心を開いて考えを変えることができるようにしましょう。何かが本当に重要かどうか自問してみてください。代替案にチャンスを与えることを意欲的に行いましょう。

コミュニケーションの取り方が重要

  • トーンを考慮する: トーンは士気に影響を与えます。それによって、コードレビューが生産的で励みになり、やりがいのあるプロセスになるか、圧迫感のある、イライラする、傷つけるプロセスになるかが決まります。親切で敬意を持ち、非判断的なトーンは、建設的なフィードバックを受け入れやすくなります。[3]  「X が間違っています」と「Y を考慮したことがありますか?」のコメントは非常に異なる効果を持ちます。
  • 意見を事実として述べない: 議論を終わらせる可能性があります。代わりに…
  • 質問をして推奨事項を行う: 文脈を提供し、コードを改善する方法を説明し、その変更がどのような影響を与えるかを説明します。説明文書へのリンクを提供すると、少なくとも一度調べたことがあることが示されます。
  • 「何を考えていますか?」と尋ね、返答に耳を傾けます。[4]
  • 何かを事実として述べる場合は、正しいことを確認してください: そうではないと、コード作成者は時間を無駄にしてイライラします。可能であれば、参照を提供してください。何かについて確信が持てない場合は、代わりに質問してください。
  • 機能的な欠陥と好みの違いを明確に: コメントに明示的にラベル付けすることを検討してください。
  • 感謝と励ましを表現する: 「ポジティブさ」は重要な用語であり、「すべてのレビューに賞賛を加える」などの助言は偽善や不要と感じるかもしれません。 代わりに、賞賛を与える機会に注意しましょう: 何か学んだり感銘を受けた場合、そのことを伝えてください。 感謝と励ましは、あらゆるレビューに追加できます: 例えば単純な「これをやってくれてありがとう」や「素晴らしい仕事だね」という +2 のコメントでも違いを生み出します。なぜなら、ポジティブなフィードバックはさらなる貢献を促し、批判的なフィードバックに対してもっと耳を傾けるようになるからです。
  • 非難や皮肉を排除する: 著者ではなくコード自体をレビューしましょう。 誰でもミスをすることがあり、成長の余地があることを忘れずに、よい協力者はお互いの成長を手助けします。 批判的で皮肉なコメントは、協力的で生産的なコード レビューにはふさわしくありません。
  • あなたが沈黙させているかもしれない人に気をつける: 否定的で絶え間ない批判の文化は、重要な声を沈黙させます。


大局を忘れずに

  • 文脈を見失わないように注意してください: 曖昧な問題や細かい指摘に集中する代わりに、コードがコードベース内で占める全体像に注目してください。自分自身に「このフィードバックは有益か?」と尋ねることで、大きな目標を考慮してください。一行ごとのレビューは重要ですが、プロジェクト全体の文脈の中で行う必要があります。あなたが貢献者を励ますことで、プロジェクト全体にとって長期的に貢献できることもあります。
  • さまざまなレビューの文脈に理解し、適応する: 経験豊富な貢献者による重要で複雑なパッチの場合、技術的な解決策を改善し強化することが主な目的になることがあります。新しい貢献者によってアップロードされた小さなパッチの場合は、許容されたり教育されたりすることが重要です。言葉遣いや批判レベルを手元の文脈に合わせて調整してください。
  • 一気に多くのコメントを残さないようにする:[5] 一度に多くのコメントを残すことは、特に複数のレビュアーによって行われた場合、著者にとって圧倒的なものとなる可能性があります。 これは一斉に攻撃を受けているような感覚を生むことがあります。 多数のコメントを残す必要がある場合、特に小さなパッチの場合、「これらのコメントが本当に必要か、価値があるか」を自問してみてください。 多数のコメントを残す必要がある場合、できるだけ著者に連絡を取り、理想的には非公開で行いましょう。助けを提供し、特に親切にしましょう。
  • 諦める: 妥協や敗北を優雅に受け入れましょう。[6] 文書化された標準の一部ではない場合は、コードの作者の好みを優先しましょう。 完璧なコードというものはないことを覚えておいてください。達成できない完璧を求めると、貢献者のやる気を奪い、進捗が遅くなります。 自分に問いかけて「このコードがそのままマージされた場合、最悪のシナリオは何か?」と考えます。 最善を尽くして、それから次に進みましょう。

思慮深い効率性

  • 明確さを目指す: ブロッカーとプリファレンスまたは明確化の要求との違いを明確にしましょう。曖昧な表現や不完全な文は避け、自分の意見や必要なことを明確に述べます。どのように衝突が解決されるかを明確にします。
  • Provide complete reviews: Review the entire patch and raise every issue at the earliest opportunity. When a new patchset is uploaded, review only the new changes. Aim to merge the code in the fewest number of review/response cycles. If you don't feel able to provide a complete review initially (e.g. if the codebase or programming language is new to you, or for a very complex patch), consider discussing the patch with the patch submitter first.
  • ...and identify when you can't: If you don't feel able to provide a complete review, consider why: is it a question of experience, resourcing, or social dynamics? If the codebase or language is new to you, or for a very complex patch, consider discussing the patch with the patch author first.
  • Stay focused: A patch should have one idea and its consequences. If you see something in nearby code that you don't like, either as a reviewer or developer, make a note or file a task, don't add more changes to the commit. Big picture or architectural discussions should happen elsewhere.
  • Avoid nitpicking: Nitpicks are comments about minor, unimportant issues that distract from the ultimate goal of the review.
    • Two developers, given the same problem, will rarely write the same code. Respect creative differences. Don't repeat the work of the developer by asking them to write the exact code you would have written. The code just needs to be acceptable, it doesn't need to be perfect.
    • When writing a review, ensure that minor issues (like code style) are not the focus.
    • Frame your comments as helpful tips, not faults to be rectified. Mark nitpicks as such and do not allow them to block merging.
  • 素早く対応する: 批判的なフィードバックは、迅速に提供され、質問や更新されたコードへの迅速な回答があればよりよく受け入れられます。
  • 自動化する: (テスト、linter、CI などで) 自動化することで、コード レビュー中の負担が軽減されます。できるだけ自動化し、繰り返しの議論がある場合は、自動化の可能性があるとして記録しておきます。

Refuse to normalise toxic behaviour[5]

  • Use your privilege: Whatever form it may take, use the authority you have to lift up your collaborators and correct or reject toxic behaviour.
  • Return to values: When you see a problem, point it out and back it up with a reference to these values.
  • Learn from your mistakes: We all have room to grow—apologize sincerely and learn from your mistake, then move on.
  • Don’t adapt to a toxic culture: We shouldn’t waste time policing how many emoji or exclamation points we use. Instead, we should question toxic cultures.
  • Get help when you need it: Contact the project maintainers or submit a report to the Code of Conduct Committee.



Acknowledgements

Many thanks to everyone who provided ideas, feedback, and resources that went into creating these guidelines.

References

  1. 1.0 1.1 Kadam, Aniket (1 April 2018). "Compassionate Coding: (The secret of high performance teams)" Medium.com
  2. Kammer, Liz; Hodges, Maggie; Murillo, Ambar (06 November 2019). "Code Health: Respectful Reviews == Useful Reviews". Google Testing Blog.
  3. Orosz, Gergely (30 September 2019). How to Make Good Code Reviews Better. The Overflow.
  4. Ciavolino, Amy (25 July 2018). A Guide to Mindful Communication in Code Reviews. Kickstarter Engineering.
  5. 5.0 5.1 Sankarram, Sandya (19 January 2018). Unlearning toxic behaviours in a code review culture. Medium.
  6. Atwood, Jeff (09 May 2006). The Ten Commandments of Egoless Programming. Coding Horror.