如何应对在code review中留莫名其妙comments的同事

职场新人，这个暑假才开始工作。

想问一下大家，有没有过同事在code review的时候抓着PR里的details或者是out of scope的东西不放，block PR的经历。

我的一个PR 因为一些NIT和一个🇨🇳同事莫名其妙的comments已经block很久了。offline聊的时候什么问题没有，笑眯眯的说这个PR没问题了。在线上review的时候疯狂comments，还都没什么价值。真的很心累。求助有经验的大佬指教，谢谢。

如果是code quality的concern，人家说的有道理，那没办法，挨打要立正，老老实实改了就好。职场新人确实得养成coding style的意识，不是能run就过关了的。

如果是那种东拉西扯让你顺便把这个干了那个干了，这个unit test顺便完善一下，那个code顺便refactor一下。那你可以push back说你把这个PR先交了再follow up那些，甚至leave几个TODO。如果人家再坚持让你现在做，你就解释一下urgency。还再继续block你那就是没事找事了。

菜而不自知 最可怕

造反精兵 发表于 2022-08-04 16:49:02
我的经验是level越高的人越不在意细节，大方向没问题就会approve。现在的组老板要大家卷coding相关的metrics，所以一堆人都疯狂没话找话。这种时候只有tech lead能帮我unblo

level越高的人不在意你的细节是因为他们忙没时间多看而且懒得管你 我合作过的人越是能力强有责任心的都会非常注重代码质量 不然直接点个approve谁不会？出了事也是你负责

只要不影响性能一律ack不就好了

匿名用户 发表于 2022-08-04 15:47:50
菜而不自知 最可怕

我确实挺菜的，我也反思过。但是comments有没有价值还是可以看出来的吧。

举个例子为什么认为没价值，不然大家也没没办法帮你

你说的detail大概率是一些code quality的问题，是很重要的

我的经验是level越高的人越不在意细节，大方向没问题就会approve。现在的组老板要大家卷coding相关的metrics，所以一堆人都疯狂没话找话。这种时候只有tech lead能帮我unblock，他说要改就改，不说的一律不改。

造反精兵 发表于 2022-08-04 16:49:02
我的经验是level越高的人越不在意细节，大方向没问题就会approve。现在的组老板要大家卷coding相关的metrics，所以一堆人都疯狂没话找话。这种时候只有tech lead能帮我unblo

有没有可能是level高的人好久不写 已经不记得细节了？

感觉有点耐心改了就是了，如果不是那种文化很毒的公司改了也能学习点东西。