ねえ!私の名前はSergeyZagurskyです。インフラストラクチャチームのJoomで働いています。コードレビュアーとしての私の実践では、レビュアーが変更をスローしてフィードバックを得ることができる魔法のブラックボックスではないことを作成者が理解していないという事実に定期的に遭遇します。レビューアは、著者と同様に、人間として、いくつかの弱点があります。そして、著者は(もちろん、彼が質の高いレビューに興味を持っている場合)、可能な限りレビューアを助けるべきです。
コードの作成者がレビュー担当者の作業を簡素化し、それによってレビューの品質とレビュー担当者の生産性の両方を向上させる方法を説明したいと思います。この記事は、レビューのために変更を準備するためのガイドとして、社内のドキュメントで使用される可能性があります。実際、それはそのようなガイドから編集されました。
なぜコードレビューを行うのですか
. , .
. , . , . . . , « », . , .
: . — . , , . . , . , , .
, . , . , , write only . .
, . - ( ). , .
— .
, , , . , . . , , « ». , , . . , . - .
, . . , . , . , . .
- , , . , , , , . , , , , , .
, -, . , , , .
. , , -, , . . .
, , , , . Pull Request .
, , , . , , .
, Pull Request, , . 500 . — 1000 . , 1000 , Pull Request’.
, , , . , , . 1000+ , , , . . , . , Pull Request’, « ».
, sed’. . . Pull Request’, , .
. , : , , , . . — . , , .
- , , . . , « » - . . , , , , .
, , . , , . , .
, . ? ! , . , . . . . - , , TODO-, , . , . .
, 10% , , . , , . , 20%, 50% , .
.
Pull Request’. , . , . «OK» «», . , , . , . — . - , , . , , , . , , . , , , .
Pull Request, , . . rebase .
? , !