想吐槽一下经理的code review

菜而不自知

菜而不自知

和我前经理一毛一样……然后是三个月他就被撤走了

size_t一点毛病没有 这个不用review CI都过不了

愿意手把手review coding style的senior role很难得的，不服的话换个单位 以后继续int i, sum写三行 😅

thegodland 发表于 2021-04-27 21:00:13
还有一点我不理解， 如果code review 加了你的名字 然后你还没review前就merge了 （另一个人approve了） 这是业界操作吗？ 还是说只是形式上加我个名字习惯了我现在也是看心情r

可能要看你们公司普遍情况是什么样吧...我们公司我看到过的大部分PR都是一小部分人approve之后就merge了，偶尔有approve比例高的都是那种一眼就能扫完的小改动，基本上没有见过所有reviewer都approve才merge的😂

lizzielee 发表于 2021-04-28 05:21:21
可能要看你们公司普遍情况是什么样吧...我们公司我看到过的大部分PR都是一小部分人approve之后就merge了，偶尔有approve比例高的都是那种一眼就能扫完的小改动，基本上没有见过所有revi

不是有required reviewers and optional reviewers吗？是不是所有required reviewers approve后就可以merge了？

两个reviewer意见冲突的时候也比较烦人。本来只要改A，reviewer1说BCD也要相应地改。我改完后，reviewer2又说不要一起改，要分几个branch单独改。我：你们俩能私下讨论好再让我改吗

TheBigJohn 发表于 2021-4-28 20:37
两个reviewer意见冲突的时候也比较烦人。本来只要改A，reviewer1说BCD也要相应地改。我改完后，reviewer2又 ...

哈哈哈 这个的确有

有规矩管得严都是好事，但要有理有据，不能独断专行或者主观臆断。我刚进组还是**的时候就是这个原则，但凡是无凭无据的主观反馈，一律怼回去，对方理亏，或者公说公有理 婆说婆有理，下次就不乱挑刺了，只说客观的问题并附上coding style guidance的具体链接