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

你可以说跟你开一个1对1的session 来解决你的concern。然后在会议开始的时候严肃的说，objective是unblock，可以merge

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

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

我在想的是这种问题你有时间跑地里来问一群没有任何context的人，为啥不直接听你manager或者mentor的。

太真实了，简直是我本人。除了是刚ramp up的senior在我pr上对service本身疯狂out of scope comment.我一个pr被block了一个多月了

其实很多事很难说有没有价值
比如我今天一个pr被block了 仅仅因为reviewer说python里不应该使用camelcase 要么snake要不全部cap
我一开始觉得没事找事  就一个变量名而已 不过想想如果他不说这个我可能会一直这么用下去

类似的经历，一大批两者都行的comment，然后PR一直block，疯狂解决冲突……我怀疑是不是和我代码冲突的那位专门找人给我comment，然后解决冲突的活就给我做😭😭😭

别人花大把自己的时间去review你的code，花费精力去留comment，你有没有想过这其实是对你的善意而不是为了block你？

反正我review code的时候关系一般的都直接点个approve，出bug了又不用我背锅，不着急自己慢慢修，着急的话就revert呗。

只有关系好的或者新人的我才会花时间去仔细看看。

也许人家为了升职正在刷comment量😂

没想到有这么多评论，感谢各位指教🙏
仔细想了想，有些comments确实是因为我水平太菜，design做的不够好。有些是我自己觉得细枝末节，但确实有改进的空间，而不是我标题说的“莫名其妙”。我很抱歉，是我标题党了。
发这个帖子的起因是我这个pr被block太久，真的很沮丧。简而言之就是被自己菜到了。我把同事给的comments全都adopt了，希望能尽快merge。
最后祝愿大家工作顺利。

匿名用户 发表于 2022-08-04 15:55:39
我确实挺菜的，我也反思过。但是comments有没有价值还是可以看出来的吧。

能不能发个没有价值的comment参考下？