张三通过邮件，知道自己的分支已经被合并进开发分支dev很开心；可以删除这个本地zhang_san分支和远程zhang_san分支，保持分支干净；也可以继续在zhang_san分支做新的开发，提交新的Merge_Request,进行新一轮的Code Review例: プロジェクト A には、master ブランチと dev ブランチの 2 つのブランチがあり、現在会社は開発中であるため、全員がブランチを dev にマージする必要があります。 ブランチ
🎜Zhang San は、dev ブランチに基づいて自分のローカル開発環境に zhang_san ブランチを作成し、関数を完成させるのに 3 日かかりました。
🎜 年老いたプログラマは、Zhang San が初心者だと感じたので、変更を dev ブランチに直接入れ、Zhang San に Merge アプリケーションを送信するように依頼しました。 コードレビュー;🎜
🎜そこで、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 を送信して、コードレビュー 。 🎜
![[Web フロントエンド] Node.js クイック スタート](https://img.php.cn/upload/course/000/000/067/662b5d34ba7c0227.png)
コード レビューは、コード レビューのシナリオを指します。新しい人または若手エンジニアが変更を加えますが、メイン ブランチまたは機能ブランチに追加する前に、経験豊富なプログラマーによる確認が必要です。古いプログラムを使わせるのは不可能です。隣に座ってコードを読みたいですか？これは時間の無駄です。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】张三通过邮件，知道自己的分支已经被合并进开发分支
🎜Zhang San は、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アプリケーションを送信するように依頼しました。 コードレビュー;🎜 🎜そこで、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を送信して、コードレビュー 。 🎜コードレビューはコードレビューです、Baiduで検索できます、たくさんの記事があります
あなたが書いたコードは他の人が読むでしょうし、あなたも他の人のコードを読むでしょう
レビュー後、再度オンラインにアクセスします
コードレビュー、一般的に大企業はコードレビューを必要とします。コードに問題がある場合、またはコードが仕様を満たしていない場合は、オンラインにする前にコードを正しく変更してください。
3,000 人の命が失われた: 古いコードの落とし穴