高洛峰2017-05-02 09:51:24
コード レビュー は、コード レビューのシナリオを指します。新しい人または若手エンジニアが変更を加えますが、メイン ブランチまたは機能ブランチに追加する前に、経験豊富なプログラマーによる確認が必要です。古いプログラムを使わせるのは不可能です。隣に座ってコードを読みたいですか？これは時間の無駄です。
Code Review, 指代码审阅；场景：新人或者初级工程师，做了一些改动，但是需要在老程序员的确认后再添加至主分支或特性分支；不可能让老程序员坐你旁边看代码吗，这样太浪费时间了；
举个例子：
A项目有两个分支master分支和dev分支；现在公司在开发，所有人完成任务，都应该把分支合并至dev分支；
张三在自己的本地开发环境，基于dev分支，创建zhang_san分支；花了三天，完成一个功能；
老程序员觉得张三是新手，不放心就这么将他的改动直接放进dev分支；就让张三提个Merge申请，让他Code Review;
于是张三就将自己的zhang_san分支提到远程服务器上，操作“New Merge Request”，发起一个Merge请求，将zhang_san合并至dev分支，指定审阅人是老程序员；
老程序员找了个空闲时间，查看这个Merge Request,觉得有点问题, 于是就提了一些建议（提的建议会自动发送到相关人的关联邮箱）；
张三后来通过邮箱查看到老程序员提的建议，在自己本地zhang_san进行了修改，并git push至远程zhang_san分支；
这时的“Merge Request”会更新最新的Commit；老程序员再看这次总的改动，发现不错，挺好的，点击确认请求后，就成功将zhang_san分支上的改动合并至dev分支啦；【到这就完成了Code Review】
张三通过邮件，知道自己的分支已经被合并进开发分支dev很开心；可以删除这个本地zhang_san分支和远程zhang_san分支，保持分支干净；也可以继续在zhang_san分支做新的开发，提交新的Merge_Request,进行新一轮的Code Review例:
プロジェクト A には、master ブランチと dev ブランチの 2 つのブランチがあり、現在会社は開発中であるため、全員がブランチを dev にマージする必要があります。 ブランチ
dev ブランチに基づいて自分のローカル開発環境に zhang_san ブランチを作成し、関数を完成させるのに 3 日かかりました。
🎜 年老いたプログラマは、Zhang San が初心者だと感じたので、変更を dev ブランチに直接入れ、Zhang San に Merge アプリケーションを送信するように依頼しました。 < code>コードレビュー;🎜
🎜そこで、Zhang San は zhang_san ブランチをリモート サーバーに言及し、「新しいマージ リクエスト」を操作して Merge リクエストを開始し、zhang_sanMerge を転送しました。 dev ブランチに追加され、指定されたレビュー担当者は経験豊富なプログラマーです 🎜
🎜古いプログラマーは時間を見つけてこの マージ リクエスト をチェックしました。何か問題があると感じたので、いくつかの提案を行いました (提案は、そのユーザーの関連する電子メール アドレスに自動的に送信されます)。関係者）🎜
🎜Zhang San はその後、古いプログラマが電子メールを通じて作成した提案を確認し、ローカルの zhang_san に変更を加え、リモートの zhang_san に git Push を実行しました。 > ブランチ🎜
🎜この時点での「マージ リクエスト」は最新の Commit を更新します。古いプログラマは全体的な変更を再度確認し、クリックしてリクエストを確認した後、それが良好であることを確認します。 は正常に完了します zhang_san ブランチの変更は dev ブランチにマージされました [これでコード レビューは完了です]🎜
🎜Zhang San は、自分のブランチが開発ブランチ dev にマージされたことをメールで知り、ローカルの zhang_san ブランチとリモートの zhang_san を削除できることを知り、とても喜んでいました。 ブランチしてブランチをクリーンな状態に保ち、zhang_san ブランチで新しい開発を続行し、新しい Merge_Request を送信して、< の新しいラウンドを実行することもできます。 code>コードレビュー 。 🎜
过去多啦不再A梦2017-05-02 09:51:24
コードレビュー、一般的に大企業はコードレビューを必要とします。コードに問題がある場合、またはコードが仕様を満たしていない場合は、オンラインにする前にコードを正しく変更してください。
