こんにちは、エンジニアの @hanhan1978です。
弊社では、ソフトウェアの開発プロセスにおいて、ほぼ全てのプログラム修正・追加に対してソースコードレビューを実施します。本番環境にはレビューを通った変更のみが反映されます。
仕組み的な話をすると、恐らく多数の会社や団体にて行われているのと同様にGitHubのPullRequestに対してソースコードレビューを行って、Approve
されないとPullRequestがマージ出来ないという形を取っています。
しかし、ソースコードレビューが常態化してくると、簡単に指摘できるような内容は少なくなってきて、「これ、あんまり良くないと思うんだけど、相手にどうやって伝えていいか分からない。」とか、「自分なら違う書き方をするし、そのほうが良いと思うけど、説得できるほど自分の頭の中で言語化できてない。」という状況によく出会うようになります。
続きを読む