【GitHub プルリクエスト】
コードレビューをする文化は根付いていますか？
自分のこれまでの実務では、コードレビューする文化はなかったです。
SIerとWeb系の違いのでしょうか。自分にはわかりません。
新人の自分が書いたコードを先輩方が確認する工数は無く、いつも納期に追われていました。。
コードレビュー無しで、push（自分はSVNを使用していたので、コミット）した結果、開発中のプロジェクトでエラーを出して、ご迷惑をお掛けしてしまうことがありました。
これを改善していくためには、プルリクエストを当たり前にした開発を行なっていく必要があるようです。
プルリクエストとは。
GitHubが最初に提供した機能であり、開発者のローカルリポジトリでの変更を他の開発者に通知するもの。
この機能のおかげで、多くの開発者がOSS開発に参加しやすくなり、結果として品質の高いコードを作ることが可能になった。
主に以下の機能がある。
・機能追加や改修など、作業内容をレビュー・マージ担当者やその他関係者に通知
・ソースコードの変更箇所をわかりやすく表示
・ソースコードに関するコミュニケーションの場を提供
プルリクエストのメリット
レビュー・マージ作業をタスク化して、やりとりを記録できる。
→ 作成されたプルリクエストを一覧で見ることができる。
→ 完了・未完了が視覚的にすぐわかるので、漏れを防ぐことができる。
→ プルリクエスト上でコメントのやり取りをして、品質向上に繋げられる。
レビューを促進できる。
→ ソースコードの変更部分を明確に表示することができる。
→ プルリクエストの作成者が、ソースコードの意図や補足事項をコメントとして伝えることができる。
→ レビュー担当者の負担を減らすことができる。
参考
参考にさせていただきました。
ありがとうございました。