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

匿名用户 发表于 2022-8-5 10:15
大部分人只会留一些empty line啊，import order之类的comments，对你也没什么帮助

这种format的事情，难道不是building的时候就应该被自动挑出来。。。。人肉去看效率也太低了

直接拿着电脑去找，当场改

stevethehitman 发表于 2022-8-4 23:05
其实很多事很难说有没有价值
比如我今天一个pr被block了 仅仅因为reviewer说python里不应该使用camelcase  ...

有一个命名的convention还是很重要的吧，其他人盲搜东西的话至少格式好猜。而且有些比较在意格式的员工看到五花八门的格式可能会抓狂

可以举几个例子吗？大家帮你分析分析

难道我理解错了吗，nit的意思就是nonblocking comment啊。我一般急着修bug或者懒得revision的时候，nit都是直接回复下个cr再改的，还能多水一个cr。至于code问题，其实你应该感谢别人帮你cr，很多人都是随便review直接LGTM然后你merge了出了问题还是自己背锅。

lxxxxxxx 发表于 2022-8-4 20:38
level越高的人不在意你的细节是因为他们忙没时间多看而且懒得管你 我合作过的人越是能力强有责任心的都会 ...

你没看懂，是老板说了psc会看每个人comment了多少别人的code，越多评价越好。

lxxxxxxx 发表于 2022-8-4 20:38
level越高的人不在意你的细节是因为他们忙没时间多看而且懒得管你 我合作过的人越是能力强有责任心的都会 ...

话不能说这么绝对 我亲历过和极端detail oriented的老人共事，那种无形的压力会让你喘不过气的  如果一个组里面的老人太多 或者政治风气倾向于排资论辈 新人通常日子不好过
这个可以在代码的review上面完美体现出来，这老人代码review光去抠边边角角的东西 关键是PR里涉及logic的没有任何评论 而是unit test逐行逐行看 给我写了20多条comment 并且还都是要求把所有用container的地方都改了用screen。。。 而且这个老人连最基本的react named import都不懂 很无语
而每当他写了一个几百行的PR扔给其他同级别senior review 反而是蜻蜓点水般一扫而过 数百行代码的pr 一句话没有就通过了。。。我对此现象至今百思不得其解