yuuki blog

プログラミング をアプトプットしています。

Git Hub ブランチの流れについて (コードレビュー)

コードレビュー

複数人での開発において、コードの記述内容に問題がないか、他の開発メンバー等が確認をすることです。また、コードレビューを担当する人をレビュアーと呼びます。

レビュー依頼の方法はさまざまです。Slack等でURL共有する方法もあれば、プルリクエスト上でSlackの様にusernameにメンションをつけることで本人にレビュー依頼の通知が行くようにすることもあります。

f:id:yuukii5:20201111184204p:plain

プルリクエストのURL(https:// github.com/<ユーザー名>/<アプリ名>/pull/1のような形になっているはずです)を共有することで、他者にレビューの依頼ができます。

プルリクエストにコメントをする

「Files changed」をクリックすると、変更修正されたコードが一覧できます。

https://tech-master.s3.amazonaws.com/uploads/curriculums//597b0a43c903769d7976fefb0fd63496.png

f:id:yuukii5:20201111184454p:plain

特定のファイルのコメントを残したい行にカーソルを当てると、青色の+ボタンが現れます。このボタンを押すとコメント用のフィールドが現れるのでコメントを書きます。以下のように記述して「Start a review」をクリックします。

https://tech-master.s3.amazonaws.com/uploads/curriculums//d7fbd5a8db3c0c6a8a47444637a7d644.png

プルリクエスト全体に対してのコメントを残すこともできます。

 

f:id:yuukii5:20201111184722p:plain

LGTM

LGTMとは、Looks good to meの略で、「コードに問題はないと思うよ!」という意味です。プログラマーが慣例的に利用します。

 

問題がなければトップページのコメントにLGTMの旨を書きます

https://tech-master.s3.amazonaws.com/uploads/curriculums//dae68b3fa1e5c41e55d8ed6b50ef35c9.png