PR之前自己的分支随便rebase
【 在 eGust 的大作中提到: 】
: 还是分干嘛,我一般情况下是用 rebase 解决 conflicts,少了一个 merge commit 完成同样的功能。逻辑上,因为代码逻辑和功能并没有实质的变化,每个 commit message 描述的变化还是相同的,所以别人并不需要知道干了啥。如果是用 merge 解决 conflicts,同样其他人也还是不知
: 道具体干嘛了,本来 merge commits 也都是次要的。
: 如果是根据 code review 更新代码,那肯定还是要有新的 commit 的,因为代码的逻辑和功能已经发生了改变。如果 commit --amend 然后 push -f,那的确不太好
说的就是这个问题
给 reviewer 带来了不方便
rebase 完加新 commit 这种还能忍,至少可以手工指定看新 commit
解决 comment 的时候在原来的 commit 直接 amend 这就太过分了……
--
FROM 220.200.25.*