Instantly share code, notes, and snippets.

@adarapata /PR.md
Last active Apr 10, 2018

Embed
What would you like to do?
Download ZIP
PRテンプレート(一例)
Raw
PR.md

困っていること

どう解決するか？

レビューポイント

@adarapata

This comment has been minimized.

Sign in to view
Copy link
Owner

adarapata commented Apr 10, 2018 •
edited

例

困っていること

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

どう解決するか？

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

レビューポイント

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

This comment has been minimized.

Sign in to view
Copy link
Owner

adarapata commented Apr 10, 2018 •
edited

補足

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