【git】基本的な改修フロー・issueからPR(プルリク)・merge（マージ）
GITのブランチについて
gitのブランチは
- master
- dev
がメインにある事が多くその後は
- feature
- bugfix
等に分かれている事が多いです。
featureは新規機能
bugfixは既存機能のプログラム修正等になります。
流れ的には
(feature or bugfix) → dev → master
の順番にコードを取り組んでいく流れになります。
PR(プルリクエスト)について
PRリクエストとは (feature or bugfix) → dev の時に反映させる前に、コードレビューをしてもらう作業になります。
gitのソース画面からpull requestを押すことによってpull request画面が作成できます。
プルリクをする前に気を付ける事
基本チェックする場所(される場所)はFill Changeの所なのでそこを重点的にチェックする事
・基本的なインデントのずれ
・いらない行を削除
・動作の流れは一通り説明できる事がベスト
・コメントを付ける（or 付けない）
最後のコメントの件についてはプロジェクトによって方針が違ったりします。コードの可読性を最大限に上げて最小限で付けましょうが現段階のベストプラクティスな気がします。
プルリク完了後、devリポジトリにmerge
devブランチをチェックアウトする
devブランチを最新にする。
devと改修したブランチをマージする。
git merge hoge_branch特定の変更だけをmergeする場合
本番環境をサクっと先になおしちゃったよみたいな場合に、dev環境で本番環境の下記のコミットのみを取り込む等もあります。
git cherry-pick [取り込むコミットID]