2016-12-11 4 views
74

機能GitHubのブログで発表し、12月、7、2016に追加、オプションを導入to add reviewers to a Pull RequestGitHubでは、レビューアと譲受人の違いは何ですか?

GitHub Review Option

あなたは今「それが簡単に誰を指定すること、協力者から明示的に審査をリクエストすることができますあなたのプルリクエストを確認したい。

プルリクエストページのサイドバーにあるレビューを待っているユーザーのリスト、および既にプルリクエストページのサイドバーからのレビューのステータスも確認できます。

しかし、PRのための校閲を設定し、明示的にはすでに人(譲受人オプション)を割り当てることによって行われました。

両方のオプションを使用できるようになった時点で、両方のオプションが同じ最終目標を共有しているため、各オプションの役割は何ですか?

答えて

63

EDIT:いくつかのOSSのメンテナと協議した後

査読は単語がすることになって何を次のように定義されます(誰かのコード)を見直し、「譲受人」するためには、緩いdefinitonについて説明しました。

「レビューア」の場合::コードを確認したい人。必ずしもその領域の責任者またはコミットをマージする責任はありません。 GitHubが自動的に提案しているように、前もってコードの塊に取り組んだ人かもしれません。

"譲受人":プロジェクトのチーム/メンテナーまで、厳密な定義はありません。それは、PRオープナー、またはそのエリア(審査が終わった後にPRを受け入れるか、ただちに終了する予定)に責任を持つ人になることができます。 GitHubは、プロジェクトのメンテナにとって、プロジェクトに最適なものを公開しておくことを定義するのではなく、

前の答え:

[OK]を、私は先に行くと、自分の質問にお答えします。書き込みアクセスを持つユーザーのPRのために

譲受人は、PRを開いた同じ人だろう、と校閲者は譲受人の選択肢のひとつ誰かであること、(コードの見直し)古い譲受人の機能を置き換えます。

書き込みアクセス権を持たないユーザ(投稿者以外):書き込みアクセス権を持つユーザは、自分自身(または他の書き込み権限メンバー)を割り当てて、PR(レビューア)を審査します。譲受人は空です。

外部寄稿者からの未完了のPR:書き込みアクセスのメンバーは未完成の仕事を引き受け、彼女のために割り当てます。彼女は仕事を終了する責任があり、譲受人です。 PRの主な理由は変更のレビューであるため、変更をレビューするために他の人を選択します。

+6

新しいチームメンバーの皆様には、譲受人と査読者の対応方法を説明するために、この回答へのリンクを送ってください。 これは、ここで根本的に何かが間違っているという考えにつながります。 –

12

GitHubでは、査読者はプルリクエストをレビューする人物です。プロジェクトの所有者はメンテナのいずれかからレビューを要求できます。メンテナが書き込みアクセス権を持っているかどうかでレビューされた場合にのみ、プルリクエストをマージできます。

公式github documentationによれば、譲受人は、特定の問題に取り組んでおり、要求を引き出す人物です。レビューアとして混乱することもある。実際には、リクエストを引き出すのではなく、問題に使用することを意図しているので、問題を受け取ったときにその問題を解決するように割り当てることができます。プルリクエストでは、譲受人は、他のメンテナーからコメントや変更要求を受けた後、プルリクエストをマージする担当者を指します。

+2

回答ありがとうございますが、私はそれが完全に質問に答えるとは思いません。誰かに問題を割り当てることができます(彼女は問題の譲受人になります)。しかし、PRが送信されると、誰かが査読者(PR譲受人)になります。この時点で、依然として譲受人とレビューア。 –

3

受け入れられた回答ごとに。はい、「譲受人」は、より緩やかな定義を持ち、チームのニーズに合わせて異なる方法で使用できます。

私たちの8人の開発者のチームでは、ほとんどのPRに1人の査読者がおり、変更を提案し、最終的にPRを承認します。審査段階では、「譲受人」はPRを開いた人物です。後で他の開発者がPRを受け取った場合、新しい「譲受人」が追加されます。 PRが承認され、QAまたは直接マージの準備が整うと、新しいQA「譲受人」が追加されます。このようにして「担当者」リストが増加します。私たちは、総称して、次の人を指定し、 "譲受人" を使用

  1. プル要求著者
  2. QA担当者は
  3. 者が責任関与(通常は同じ1のように)PRの変更の提案に取り組んで
  4. 著者(通常は2または3と同じ)

"譲受人"を使用すると、将来のeasでのPRの位置を特定できますily。私のプロジェクトの1つが3000を超えるPRを持っています。検索プロセスを容易にする

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

および他の同様のクエリ。 「マイルストーン」は、PR検索を簡単にするのにも非常に役立ちます。

Screenshot Github, Q4 2017

関連する問題