我们用 bitbucket,comment 是基于文件的,只要文件还在,就还是能看到评论。如果文件不在了的话,就只能点到每一个 commit 里面去看了。如果 comment 在一个新文件里,之后改代码把文件干掉了,那 -f 就看不到了……如果 comment 该行的代码变了,那就会跑到 outdated comments 里面去,这点跟 -f 没关系。
不确定 github 逻辑啥样
PR 的大小也是一个需要控制的点,一个 PR 如果改了超过50行复杂代码,我个人感觉就已经超过能通过 review 检查出问题的上限了。不 pull 下来整个 branch 跑一下的话,能做的就只有检查一下代码风格,功能在大方向有没有问题了。换句话说,如果一个 review 一个 PR 搞出了20条 comments,那回头代码变了,再检查20个点有没有都改到也是很浪费时间的
【 在 dpblue 的大作中提到: 】
: -f 会不会把别人之前的review comments给冲掉?
--
FROM 203.211.104.*