一个PR在merge之前commit很多次会有什么负面影响吗？

今天看到地里有个被胡萝卜pip的同学说被pip的原因之一是这个。赫然发现自己也有这个问题。。。


补充内容 (2021-10-10 09:50 +8:00):
谢谢大家的回复。
我一般就做的是以下四步，现在经大家提醒以后再加上个git squash. 请问git还有啥其他的基本常识吗？万分感谢！
git checkout -b branch
git commit branch
git rebase master
git push --force origin branch


补充内容 (2021-10-17 11:16 +8:00):
感谢大家的回复。过了一周，现在我所有的merge 到master都用squash and merge. 小的改动经常用git commit --amend.

没有任何影响，胡萝卜那个是欲加之罪何患无词。

git squash可以了解一下 😄

我都是10+，每改一行就commit一个，生怕没保存白改了，只要不让peer反复review我觉得就没事

git 有个功能 可以把多个commit最终合成一个commit check-in。
每次改动都应该commit，不然丢了咋办

我改个variable 就commit一次(需要测试)，要发pr时git reset到master版本然后commit一下就一个commit了

review之前加上[WIP]
多人review的话resolve所有comment之后一次性修改
最忌讳反反复复，变来变去

小亩_2hq1okp 发表于 2021-10-9 21:19
git squash可以了解一下 😄

这个对于已经merge的PR有用吗？

请问git rebase master是什么作用？

rongmeimei 发表于 2021-10-9 21:53
请问git rebase master是什么作用？

就是有时候你的branch落后于master了，要rebase一下。不然有的test过不了。我其实不是很了解ci的原理。因为change set还是一样的。为啥了rebase test就能过，不rebase就不行。