Skip to content

Instantly share code, notes, and snippets.

@adarapata

adarapata/PR.md

Last active April 10, 2018 09:30
Show Gist options

  • Select an option

    Learn more about clone URLs
  • Save adarapata/40ec5f66e0c348a639a1aa6cb519aee6 to your computer and use it in GitHub Desktop.

Select an option

Learn more about clone URLs
Save adarapata/40ec5f66e0c348a639a1aa6cb519aee6 to your computer and use it in GitHub Desktop.
Download ZIP
PRテンプレート(一例)
Raw
PR.md

困っていること

どう解決するか？

レビューポイント

@adarapata
Copy link
Author

adarapata commented Apr 10, 2018 •
edited
Loading

補足

  • 「困っていること」はこのPRをやる意味が書いてあると、レビュアーの納得感(これを解決できるの最高だな！ならレビューしないと！)を底上げできる。
    • 不具合改修は困っていること書きやすいけど、機能追加はサッと書くの難しいことがあるので時間かけすぎない範囲が良さそう
  • 「どう解決するか？」はどういう方向で上記の問題を解決したか、くらい抽象的だとレビュアーが突っ込みやすい
    • 今回だと「銃弾の速度を下げる」「壁を厚くする」などのアプローチ自体の提案をしやすくなる(良い悪いは別として)
  • 「レビューポイント」はもっと良くできるんじゃないかというところや、肝の部分だから理解して欲しいところなどを書くと、レビュアーが集中して見れる
    • 逆にこの部分はこんな事情があるので今回は見なくて大丈夫です！みたいなのも書いたりする

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment