私は比較的人気のあるgithubリポジトリを維持しています。
プルリクエストがマージに適している場合、通常、マージする前に作成者に単一のコミットにリベースするように依頼します(特に複数の小さな編集がある場合)。
これは良いGitプラクティスですか?これは許容できる/標準のGitHubエチケットですか?
いくつかの利点:
- コミットログにきれいなコミット履歴があります
- 自分でコミットを変更する必要はありません
- 作業の一部を委任します
考えられる欠点:
- これが良いエチケットかどうかわかりません
- これが良いGitプラクティスかどうかわかりません
- 私は通常、すでにいくつかの他の変更を求めています-これはもう1つであり、貢献者を落胆させたくありません。
1
この方法でプロセスを実行する際に見られるいくつかの利点と欠点を説明できますか?
—
アレックスファインマン14年
考慮に値するいくつかの追加の利点と欠点。良い:コミットごとにビルド可能な状態または完全な状態が生成されると、git-bisectおよびその他の反転が容易になり、このアプローチはそれを保証する簡単な方法です。bad:単純なコミットメッセージによる小さな変更がメガコミットにロールバックされます。たとえば、「この1行を変更してコーナーケースをまあまあ修正する」は、「機能fooの追加、変更の大きなリスト」に組み込まれる可能性があります。これにより、特定の変更の理由を見つけるのが少し難しくなります。
—
ガンクロ14年
標準の設定に問題はありません。予想されることを前もって明確にしてください。例:symfony.com/doc/current/contributing/code/patches.html下にスクロールしてステップ3:パッチを送信します
—
Cerad 2014年
@Granko:「リベース」と「単一コミットへのリベース」は2つの別個の問題です。
—
マシューシャーリー14年
貢献者がこれを行うように求められた場合、プルリクエストのブランチを
—
フリム
git push -f
?で上書きする必要がありますか?