Github で上で PR をレビューしている方も多いと思います。 その中で、よくあるのが typo への指摘です。 Typo を指摘する際、
`type` は `typo` の誤りではないでしょうか?
というような形で指摘することになると思うのですが、これをやっていると
- reviewer が上記のようなコメントを書く
- author が上記のコメントを元に修正コミットを入れて push する
- reviewer が accept する
という不毛なやりとりをしてしまうことになります。メンドい。
こういうケースにおいては、Github の suggested changes 機能を使用しましょう。
suggested changes 機能とは
suggested changes 機能は、Reviewer が PR のコメント欄に特定フォーマットでコメントすることで、author が Github の UI 上でボタンを押すだけでその内容を取り込めるというものです。
フォーマットとしては、以下のように suggestion ブロックとして記述するというものです。
すると、Author 側には、Reviewer からの suggestion をクリックして取り込めるボタンが表示されるようになります。
上記のボタンだと、押下するごとに commit が行われる(コミットメッセージは入力できます)ようになりますが、複数個の suggestion を一括して取り込みたい場合は、以下のボタンで batch に suggestion を追加していき、最後に一度の commit でそれらの修正を取り込めるようになります。
これにより、Reviewer 側の「こんなに小さいのを指摘して直してもらうのちょっと気が引けるなぁ」というような引け目を感じなくすることができて、個人的には便利でした。
