web-dev-qa-db-ja.com

2人のチームのプルリクエストの紹介-自分のリクエストをマージしますか?

ジュニアチームメンバー(生協)にgitを紹介しています。

追加、コミット、プッシュ、プルの基本的な操作に慣れています。

次に、プルリクエストとブランチにそれらを紹介したいと思います。

ブランチでプルリクエストを実行し始めた場合、進行中の作業にも同じことをする必要がありますか
私は彼らのプルリクエストをマージします。ブランチで作業することが最も理にかなっているかどうかはわかりませんでした(通常、私は知っている良い習慣ですが、この特定の状況に興味があります2人の開発者と1人のジュニア)。そうであれば、自分のブランチをマスターにマージするだけです。とにかく、自分の作業/ブランチのプルリクエストを行うこともできますか?通常、これらの変更には基本的なgithub機能分岐ワークフローを使用します。
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

自分が唯一の開発者である場合、自分のリポジトリでプルリクエストを使用する目的はありますか? は便利ですが、それほど具体的ではありません。

プロジェクトの2人でのワークフローはどうですか もより一般的なようです

そして

公式リポジトリまたは私のフォークのブランチからプルリクエストを開く必要がありますか? フォークについての詳細のようです。

11
Michael Durrant

いいえ。自分のプルリクエストをマージしないでください。ガチョウにとって良いことは、ガンダーにとっても良いことです。独自のプルリクエストをマージすることは、私たちのジュニア開発者にとって悪い前例となります。また、他の誰もあなたのコードを見ているわけではありません。私たちがどんなに年上であろうと、私たちは皆間違いを犯し、時々悪いコードを書きます。ジュニアにあなたの仕事をレビューさせ、マージさせることにより、反対側からコードレビューがどのように機能するかを教えます。

彼はあなたと同じ目を持っていないかもしれませんが、レビュー担当者の側からプロセスに慣れるでしょう。最低限、それはあなたには明らかであるが彼には明らかではないコードの断片を示すでしょう。これには2つの利点があります。

  1. どちらも、ジュニアが学習活動に集中する必要がある場所を学びます。
  2. 自分が本来あるべき場所よりも賢い場所を学びます。
19
RubberDuck