Skip to content

Instantly share code, notes, and snippets.

@adarapata
Last active April 10, 2018 09:30
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
Star You must be signed in to star a gist
Save adarapata/40ec5f66e0c348a639a1aa6cb519aee6 to your computer and use it in GitHub Desktop.
PRテンプレート(一例)

困っていること

どう解決するか?

レビューポイント

@adarapata
Copy link
Author

adarapata commented Apr 10, 2018

困っていること

たまに銃弾が薄い壁を貫通してしまう。これだとゲームバランスが崩壊するので直したい。
銃弾が高速で飛ぶので、銃弾のColliderが薄い壁のColliderをすり抜けているっぽい。

どう解決するか?

事前に正面にRayを飛ばして衝突を検知できるようにした。

レビューポイント

Rayを飛ばす処理がコスト的にちょっと不安なので、良い方法があれば教えて欲しいです!

@adarapata
Copy link
Author

adarapata commented Apr 10, 2018

補足

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

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