他人のコミットをgit merge --squashするべきでないのではという話 - Qiita

最近某OSSに出されたPRが、git merge --squash <branch> でマージされたことにより、コミットのAuthorが書き換えられてしまったことが一部界隈で話題になっていました。この件にはマージを行った人に悪意はなかったようなのですが、gitの理解不足により生じてしまった案件だとすると悲しい話なので一応メモ 何が起きたか コミッターが複数の内容が含まれたPRを送った 管理者はその中の一部の内容だけをマージするために、管理者はgit merge --squashを実施し、コミットを改変した上でmergeを実施した ←これが問題 コードの内容はコミッターのものなのに、Authorだけ管理者にすげ変わってしまいコミッターのモチベを損ねた そもそもsquashするとどうなるの ここに分かりやすくまとまっています。 アジャイルSEを目指すブログ 図で分かるgit-mergeの--f

[megumin1]

PR上の会話を見る限り、無関係なものを含む大きいPRをいきなり送ったり、メンテナが懸念を示したにも関わらずPRをわける努力もしないで逆にメンテナにcherry-pickを要求したり、明らかにPRを出した側の問題が大きいでしょ

[tomo_ari]

Git は author と committer を両方保持できるんですよ

[yug1224]

悪意があったわけでも無さそうなのに寄ってたかって👎するのスマートじゃないね

[tmtms]

GitHub作法めんどくさいな。まあ気持ちはわからんでもないけど。

[razokulover]

これの話か https://github.com/Microsoft/winfile/pull/42

[junjun777]

ログが多少汚れても気にしない方が良い。squashはAuthor以外はすべきではない。失われる情報が多すぎる。

[diveintounlimit]

言いたい事は分かるけど、後で追いかけづらいからmaster宛はsquashして欲しいなぁ(´・ω・｀)100個単位の「インデントが間違ってた」とか「変数名調整」のコミットはあんまり見たくない/しかし当該PRは見るに堪えないな…

[Magicant]

Git はあくまでも版管理ツールであると思へば、コミットに committer を記録する意義はあっても author は記録する理由がないのではと感じる

[slash_01]

社内で開発するときはそこまで気にはしないな。基本はコード書いた人にrebaseもmergeもしてもらうけど。 ソーシャルな世界でNGなのは分かる。

[hase0510]

merge --squashやcherry-pickで怖いのは必要なコミットのマージ漏れ。原則としてすべてのブランチは最終的にmasterに合流させる。不要なコミットはrevertして不要だという判断を明示的に歴史に残してからmergeする。ってしてます。

[shoh8]

コントリビュータをお粗末にしたらOSS死ぬもん

[luccafort]

これ元々の発端を起点にして「じゃあ実際そういうことが起こったらどうするのがベターなのか？」というのが書いた人の意図だと思うんだけど最初の事例に意見引っ張られすぎてないか？

[gabari]

OSSコントリビュータにとってCreditは非常に大事だと思うので、単なる気持ちの問題ではない。

[ledsun]

GitHubのsquash and mergeだとcommitは残ったと思うけど、authorは気にしたことなかったなあ

[masa8aurum]

squash すると author が変わるらしい

[sotonohitokun]

初見プログラムネタと分からずルー語(英単語多用系日本語）と思って読んでイミフだった。

[koogawa]

Appleさんにsquashされた事あるけど、あの ときは納得感があった

[masudaK]

ケースバイケースだろうけど、お作法考え方の違いもあるし難しい

[yusuke-k]

PRの一部のコミットだけ取り込みたいなら、普通にcherrypickするだけでいいのになー、と読んでて思いました。

[lazex]

他人のは基本書き換えないけど、書き換えられたくらいでこんなに一々文句つけてるとか面倒な人もいるもんだなぁ。プロジェクトが良くなったのならコミットのauthorとか誰でも良いよ

[Cherenkov]

mattn winfile

[wata88]

コミュニケーションは難しい。gitの特定の操作が難しいという部分もあるとは思うが。

[daichirata]

この辺ツール側にもうちょい改善の余地ありそう

[igrep]

rebase --interactive