ブランチ切って、何度かコミットして、プルリクエスト送って、マージされるまでの流れがわからない

[pss-aileen]

• ブランチを分けて作業をする際、こまめにコミットをしてログを残す...？
• そのコミットメッセージはどのように書けばよいのか←ここがわからない
• 1issue、1commitとするためにはどのような動きをすれば良いのか？
  • git rebaseとか...？
• 一連の流れの例がほしい。

---

以下試行錯誤メモ

git reset --hard
git reset --soft
git reset --mixed

などがあったり
プルリクエストのマージの時に
Squash and merge なるもので一応1コミットにできた
Test by pss-aileen · Pull Request #2 · pss-aileen/practice-github

[kazzyfrog]

1つ簡単なイシュー取り組んで、PR出してみました！
#14

コミットの粒度などは、初めはそんなに気にしなくても良いのかなとも思いました
なので、細かくコミットして複数になっても気にせずOKです！

[pss-aileen]

ありがとうございます！！

私は一人でするときに途中経過すらもコミットしているところがあって、
コミットの粒度がものすごく小さかったんですよね。

この共同プロジェクでは可能であれば、 @kazzyfrog さんがされたように、
1つのコミットにまとめて？ or 作業が終わってからコミット、プルリクを出すといった感じで良いでしょうか？

[pss-aileen]

あと、プルリクにあたってのテンプレは kazzyfrogさんのを参考にやってみようと思います

## 概要
## 関連するIsssue

[pss-aileen]

質問攻めで申し訳ないのですが、レビュー機能？

こちらは基本的に、誰か一人がレビューしないとマージできないといった感じでしょうか？

そして、マージは基本「Merge pull request」で良いのでしょうか。
SquashとかRebaseなどもありますが...

[kazzyfrog]

1つのコミットにまとめて？ or 作業が終わってからコミット、プルリクを出すといった感じで良いでしょうか？

これは、１つのissueで１つのタスクなら、１コミットにまとめて良いと思います！
１つのイシュー内に、以下のように複数のToDoがあれば、タスクごとにコミットを分けたほうが見やすいかもです：

• taskA
• taskB

[kazzyfrog]

こちらは基本的に、誰か一人がレビューしないとマージできないといった感じでしょうか？

これは、リポジトリの設定で、レビューが必要がどうか、最低何人のレビューが必要か、などの設定ができます。
現在は、1人以上のレビュー・承認がないとmainにはマージできない設定になってます！
よかったらレビューしてみて下さい！
今回はテンプレ通りに作成したファイルなので、メッセージは「LGTM」でOKです

あと、マージは基本「Merge pull request」でOKです！

[pss-aileen]

ログ...

---

今回はApproveかな...

---

Approveした...！

---

---

[pss-aileen]

まとめ

• ブランチ作る
• 1つのissueに1タスクであれば1コミットにまとめる ref
• プルリクエストを出す ref
• 誰か一人がレビューする 手順
• Merge pull requestでマージする

[pss-aileen]

@kagomen @kana1166
実際のブランチ切ってからプルリク、マージするまでの流れです！！
参考までに共有です！