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

review code还挺累的，只要愿意给Comment，我都觉得栓Q

卷死ta， 找找你们公司的coding standard，convention，guideline 之类的，肯定有一堆没人遵守的，然后熟读并背诵，以后看到他的提交就按照guideline 一个一个挑刺，并附上guideline 链接，你看我不是想找你麻烦， 我是按照规则做事。

匿名用户 发表于 2022-8-5 12:11
卷死ta， 找找你们公司的coding standard，convention，guideline 之类的，肯定有一堆没人遵守的，然后熟读 ...

wait。。。这不是大家应该做的吗，难道你觉得这个行为是在恶意挑刺？？？？？

讲道理，真的只有关系很好，或者组里的TL（工作职责所在）愿意读你的代码和文档。大家都是打工人，真没人吃饱了撑的给你挑刺的，都想早点下班

你真是身在福中不知福，能有组员及时给你review code，那是人家负责，积极，这还不感谢人家，我这边都是求着别人看，不催就没动静

奚仲 发表于 2022-8-6 00:22
wait。。。这不是大家应该做的吗，难道你觉得这个行为是在恶意挑刺？？？？？

很多大公司convention很多，甚至许多有冲突的，多到不可能所有人都100%遵守。 我经常见到 api review committee自己都忘了有这么个规定然后被人反将一军的。

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

大部分人只会留一些empty line啊，import order之类的comments，对你也没什么帮助

如果你完全理解了那个comment, 并且有足够理由认为那个comment没必要，你就在他comment下回复，然后resolved他的comment. 一般reviwer让我改什么我就照着改，除非他推荐什么fancy的code style给我，我会拒绝。或者说PR很急，那就在他comment下面再开个ticket说，下次解决。

lz我特别理解你，我code quality可以说挺高的，之前公司我的code和OOD都是被当成template给别人抄的，TL都会在public channel宣扬我PR质量高这种（加前提防杠）。

今年换了一个startup，本身repo质量就差到无法忍，还遇到了一个和你说的一模一样的case的人，真的是超级低质量的comments，疯狂block我的PR两周，我天天催，都催到快崩溃了。平时这个人也是笑面虎，还经常夸我，根本看不出来背地里阴戳戳地block别人PR。说真的，工作三年多了，啥样的人都或多或少遇到一些，这种人还真的让人头疼没特别好的办法处理。

高赞我也点赞了，说的非常make sense，基本上decent的reviewer都会在code基本ready，你自己也fully test的时候，先approve去unblock你，最多会让你加点TODO（说实话加TODO这种人也少见）。

我不是想给你出什么主意，就是想安慰一下你，别被评论区很多戾气很重的讽刺打击到了，有事没事找事block你的code reviewer不是你的错，他们自己的毛病。

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

empty line本来就该删 而且我身边的人都不会只提这些 你可以试着换个程序员质量高的组/公司