web-dev-qa-db-ja.com

貢献者にgithub上のプルリクエストをリベースするように依頼することは正しいですか

私は比較的人気のあるgithubリポジトリを維持しています。

プルリクエストがマージに適している場合、私は通常、作成者にマージする前にそれを単一のコミットにリベースするように依頼します(特に、複数の小さな編集があった場合)。

これは良いgitプラクティスですか?これは許容できる/標準のGitHubエチケットですか?

だからいくつかの利点:

  • コミットログにニースのクリーンコミット履歴が表示されます
  • 自分でコミットを変更する必要はありません
  • 一部の作業を委任します

考えられるいくつかの欠点:

  • これが良いエチケットかどうかはわかりません
  • これが良いgitプラクティスかどうかはわかりません
  • 私は通常、他のいくつかの変更をすでに求めてきました-これはもう1つの変更であり、寄稿者を落胆させたくありません。
25
Pablojim

Gitに関する限り、ブランチをマージするだけなのか、マージしているブランチの最新バージョンでコミットをリベースするのかは、聖戦のようなものです。 Programmers.SEのクイック検索 を実行すると、どちらが良いかについての会話がたくさんあります。

その背後にあるエチケットに関しては、これを実用的な観点から扱いましょう。他の誰かからの新しいコードを処理するときは、ブランチからの最新の変更をマージするか、マージする前に新しくリベースして、クリーンなマージを確実にするのが最善です。覚えておいてください、彼らはコードを書いたので、それらは通常、マージ/リベースの競合に対処するのにはるかに最も適任です。個人的には問題はないと思いますし、他の人からのリクエストもいつも見ています。私にとって、競合がない場合は、gitが適用できる2秒の更新であるため、自分で行うことがよくあります。ただし、競合がある場合は常に、コードの元の作成者に自分で対処するように依頼します。

また、GitHub(少なくとも)の場合、具体的には、PRの試行の前にCONTRIBUTINGファイルへのリンクが表示されるため、期待の概要をまとめるのに適した場所になり、多くのプロジェクトでは、今日までブランチ。

16