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

匿名用户 发表于 2022-05-29 14:02:54
哎……咋说呢。这个同事如果对所有人都这样，那楼主就睁一眼闭一眼得过且过。大家都能忍，那你也忍。

如果这个同事针对你，那我建议你写个internal的wiki（visible to all），基本

我也这么建议开会，但大概率我是junior，对方直接拒绝写doc，然后会议结束立马恶心我给我找了7，8处nits……真的绝了，我也告到老板那里。最后老板和稀泥，真的觉得有些人真的是技术有限，一天天只能找些细节review，真让他找点logic问题半天找不出来。将来我review别人的code，是绝对不会让nits阻碍别人code approval

所以每次别人找我review，我其实不懂她在干什么，我也没有context，也不懂那部分code base，纯粹因为她找不到别的人了。

结果我就直接一个approve/lgtm

这样其实大家都开心哈？

如果对方给的是LGTM with nits，对方是expect你要么改了这些nits，要么解释下为什么你觉得不用改。如果是悄咪咪的就直接merge了是很不professional的。

有些小的东西直接按reviewer的意见改完了不是比跟reviewer argue更快？除非有些code style你有不同的意见，那可以说this needs a broad discussion。
职场很重要的一点是对事不对人。

这就是为啥大家一定要拼老命push自己赶紧升到senior，一来是这样的傻叉不敢轻易block，二来是可以更好地帮助junior组员完成他们的工作

torf1018 发表于 2022-5-30 06:33
有些小的东西直接按reviewer的意见改完了不是比跟reviewer argue更快？除非有些code style你有不同的意见， ...

这取决于所谓的“小的东西”有多小，以及reviewer自身的素养。
我是遇到过好几次做一个很小的css改动被要求做refactor，结果turned out多达10+个file的change的

也遇到过一个reviewer看完，提了一些小东西+block；改完后第二天他再回来看，又提了些小东西+block；改完后第三天他don't feel well请了两天假，又因为被block住，别人也没办法approve。结果在老板看来，我一个css change做了一个礼拜，得亏我司没有pip，不然我早死了。

从此以后对这种吹毛求疵的reviewer格外警惕。没办法，不是我不想对事不对人，deadline在那里压着

试着跟senior沟通一下, 跟manager 1-on-1的时候沟通一下, 提出你的观点, 看manager怎么说. (一定要强调这些东西做到完美很花时间, 但是收益并不大, 会导致整个组的绩效变差, 这才是mgr的痛点). 另外也私下问问其他组员的看法, 如果他们跟你的看法一致, 可以联合起来说, 比你一个人讲效果好很多.

话说review标点符号之类的, 你们组得有多闲才会看这些. 我平时review代码, 只会看结构性错误, 业务逻辑, 影响性能, 可靠性, 可观测性之类的东西. 代码命名, 写法之类的只要不影响阅读, 即使不符合我的风格我也不会干涉, 最多comment一下说这个是very minor的, 改不改你自己决定.

一个senior, 目光应该放在更大的地方, 纠结细节, 本身就影响自己的格局.

torf1018 发表于 2022-5-30 06:33
.1point3acres有些小的东西直接按reviewer的意见改完了不是比跟reviewer argue更快？除非有些code s ...

没错，真的懒得argue。

狗家的readability吧。
-baidu 1point3acrescomments结尾都要加标点。

就事论事，有则改之，无则加勉。