web-dev-qa-db-ja.com

誰かのプルリクエストを編集するためのエチケット

私はGitHubにレポジトリを所有しており、1つのコミットでプルリクエストが送信されます。私は彼のソリューションを部分的に実装したいだけで、ユーザーが行ったコード変更の約半分を使用しています。この状況ではどうすればよいですか?

彼のバージョンのブランチを作成し、戻って、元のバージョンから保持したい「古い」コードをコピーして2番目のコミットに貼り付けます。これにより、コミット間の差分が大きく見える場合があります。それは本当に、そしてgit blame

私が彼のコミットから保持したいコードをコピーして、新しい別のコミットに貼り付けます。これは、彼がコードへの彼の貴重な貢献のクレジットを受け取っていないことを意味します。

上記と同じように、彼のコードの一部を新しいコミットにコピーしますが、コミットの作成者を私ではなく彼に変更します。彼は技術的にはコミットされたexactコードを記述しなかったため、これが不快に思われるかどうかはわかりません。しかし、少なくとも彼は、使用されているこれらの行の属性を取得します。

20
IQAndreas

私はオプション4でいきます。プルリクエストがプロジェクトの目標に適合しない理由をコントリビューターに説明します(そしてその過程で、コントリビューターがそれを考える理由を説明する機会を提供しますdoes)。彼は、プロジェクトに適合する変更のみを含む新しいバージョンを再送信します。

これには3つの利点があります。

  1. あなたは仕事をする必要はありません;-)
  2. 寄稿者はあなたの目標をよりよく理解し、将来の寄稿では変更が不要になる可能性が高くなります
  3. 所有者の感覚を保持することに加えて、貢献者は有用な貢献をしたと認められ、認識されます。それらのすべてが彼らが貢献し続ける可能性が高くなります
36
Jörg W Mittag