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

对方故意给你comment让你改，然后每次你改完再发新的小东西，很容易无数commit了，不要问我怎么知道的

我之前也是太多commit，被mentor教育了，然后同组的印度小姐姐告诉我，可以用git commit amend

原来如此

如果用github的话 merge PR的时候不是可以 squash and merge么 用那个就可以了

htd2020 发表于 2021-10-10 14:59:04
对方故意给你comment让你改，然后每次你改完再发新的小东西，很容易无数commit了，不要问我怎么知道的

这种人是什么心态？还有就是多个reviewer有时意见不太同 听谁的呢？ 三个reviewer 前两个当天给我review了 然后我解决了comments发了一个push 然后其中一个reviewer approve了 然后第三个reviewer出来写comment 然后我再push 之前的approve也被reset了 又要重新approve

pangpang1234 发表于 2021-10-12 00:22:22
我之前也是太多commit，被mentor教育了，然后同组的印度小姐姐告诉我，可以用git commit amend

就是第一次commit后 后面所有的修改都不断amend到第一个commit中？这个和squash什么区别？

油冬冬 发表于 2021-10-20 22:58
这种人是什么心态？还有就是多个reviewer有时意见不太同 听谁的呢？ 三个reviewer 前两个当天给我review了 ...

有的是要刷存在感，抢visibility，有的就是要整你

自己的dev branch爱怎么搞怎么搞，要发PR了搞个新的git merge --squash，又是一条好汉
当然没事连这都管的公司也是绝了

合并一下commits，想留几个留几个，具体查一下rebase和squash的用法就好，git永远的神！我现在push代码去的一个repo被要求一个pr只能保留一个commit，每次都要这么搞的