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


11

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

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

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

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

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

プロジェクトに2人での作業の流れはいただきました!また、より一般的なようです

そして

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

回答:


19

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

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

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

6
コードレビューのもう1つの大きな利点は、コードの変更を行う前に、少なくとも2人がコードの変更を1つ1つ見て、知っており、質問する機会があったことです。そのすべてから何かを学ぶことが保証されています。
Ixrec
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.