求问如何对待吹毛求疵的code reviewer

匿名用户 发表于 2022-05-29 07:45:05
楼上都这么心胸大度吗？。。

我也有个同事这么吹毛求疵，每次code review评论一堆有的没的。。

你这个还可以，我贴了doc，我那个大哥直接跳出来说doc是outdated，要以他说的话为准……就很奇葩，就很明显欺压junior engineer，然后老板明明心知肚明，还照样打压我说这是让我learn……跑路中

匿名用户 发表于 2022-05-29 09:57:46
你这个还可以，我贴了doc，我那个大哥直接跳出来说doc是outdated，要以他说的话为准……就很奇葩，就很明显欺压junior engineer，然后老板明明心知肚明，还照样打压我说这是让我lea

是，有时候碰到这种奇葩同事跑路真的是最优解。我这人没啥特点，就是喜欢正面刚。

我现在也打算跑路，我现在这公司也是奇葩，有一个TL, 在这公司呆了四年，之前印度空降，半年前和我老板同时lead两个project。

我和我老板都是new hire, 结果现在我们的project做完快一个月了，这TL天天使唤我给他们的project修bug. 我直接就ticket description 里详细描述这个bug是哪个ticket造成的，然后把ticket assign回去。搞了一两次把这TL搞炸了，直接slack里说不许assign回去

开会的时候还质问我为啥assign回去。我直接说I didn’t write this logic, and the bug is caused by ticket xxx as far as I can tell. 然后这货又自顾自说什么next time please let me know before you assign the ticket back. 我直接说I didn’t write these logic, and did I ask you to fix xxx bugs for me?? Xxx是我写的project. 这TL就什么we need to be like a team. 🙄️

我现在这公司好多比较复杂的业务逻辑，我看git blame, 再搜那些作者全是inactive account… 感觉像是刚经历过大清洗。。留下的都是政治斗争的胜利者🤣

匿名用户 发表于 2022-05-29 11:00:46
是，有时候碰到这种奇葩同事跑路真的是最优解。我这人没啥特点，就是喜欢正面刚。

我现在也打算跑路，我现在这公司也是奇葩，有一个TL, 在这公司呆了四年，之前印度空降，半年前和我老板同时lead两个pr

一模一样，一堆shit……比如我们组那个总挑我毛病的最近搞了一大堆memory leak，当初用错误的方法指导我，我怼回去这个不对，他还跑去告我老板说我不愿意学习……无论我怎么解释，老板肯定觉得junior水平一般，老人是对的……这下large memory leak 我看他下周怎么办………归根到底就是菜鸡互啄

匿名用户 发表于 2022-05-29 11:00:46
是，有时候碰到这种奇葩同事跑路真的是最优解。我这人没啥特点，就是喜欢正面刚。

我现在也打算跑路，我现在这公司也是奇葩，有一个TL, 在这公司呆了四年，之前印度空降，半年前和我老板同时lead两个pr

赞一个 老哥不卑不亢 有理有据！

你这reviewer可能是狗过去的

如果是大家都按这个 senior 说的这么写，那么就按照他／她说的，如果别人没有，那么就说:
- it seems like a personal preference than actual benefit, I also saw this xyz example in the code base. If we want to insist a common practice, let's have a broad discussion with the team and we will all do a follow up PR/CR to fix it. But let's have a team discussion first.

这种挑刺的最遭人恨的。为了避免成为这种人，我遇到别人让我review代码，只要那人写的是代码，而不是武侠小说，我就直接approve。

哎……咋说呢。这个同事如果对所有人都这样，那楼主就睁一眼闭一眼得过且过。大家都能忍，那你也忍。

如果这个同事针对你，那我建议你写个internal的wiki（visible to all），基本就是screenshot 所有他提出的芝麻问题，贴到wiki里，title就叫-- Memo/manual for code review
记得也多贴几个reasonable有用的code review from other coworker。

如果他再惹到你，就开一个组会meeting（一定放在周五下午，说完就闪） -- title就叫 code review  brainstorm。正面刚，go through这个wiki，问问大家还有什么需要加和需要考虑的。我曾经就被针对（连特么comment # 多打了个空格都让我改，然后呢那个姐妹儿改了一处，然后说让我自己去找剩下的……）。后来开了组会，我做了个详细的ppt之后，那个恶心人的姐们儿就拒绝给我做code review，我找老板换了个正常哥们儿，就没事儿了。

对待别人的评价有则改之无则加勉，我觉得这挺正常的啊

ConnieQ 发表于 2022-5-29 17:05
对待别人的评价有则改之无则加勉，我觉得这挺正常的啊

祝福你遇到这么能有的coworker