Code review故意找一个容易过的人看？

这问题太常见了。我也遇到过teammate CR故意绕开我，或者design review我提了一些问题就没有被邀请到follow up discussion的情况。我觉得我的要求其实不太苛刻，并且如果他们有好的理由我一般也不会固执己见。
Junior一般这么做要么是图省事想要快速deliver。或者就是真的“害怕”跟你argue，因为会很尴尬很打击自己。或者就是你真的很push很强词夺理但自己并没有意识到😂
观察到这种现象的时候我一般会schedule一个casual 1-1，不要盲猜为什么，直接1-1跟他们聊，然后对症下药。
和前面某位同学说的一样，pair programming也是个好方法，comment太多的时候不妨跟他直接来个meeting。

让他一小段一小段比如20-50行的check in，这样你的comments可能就不算多了。

我们review也有个很严格的人 毫无意义的严格
我们走了一样的route，不过人家的是另一个人review通过了，到了他这里就觉得那个不合理，让我们改我们的+他们的。 久而久之我们也不找他review了。
如果说review标准不一样， 请reviewer内部解决 而不是莫名其妙让我们后来的躺枪

Ruby920421 发表于 2022-04-20 09:06:23
这问题太常见了。我也遇到过teammate CR故意绕开我，或者design review我提了一些问题就没有被邀请到follow up discussion的情况。我觉得我的要求其实不太苛刻，并且如

受教，也许是个值得尝试的方法！我就是觉得别人故意不叫我，明明知道我是service owner，而且之前的代码都我写的，我还去主动1:1会不会有点奇怪？

匿名用户 发表于 2022-04-20 09:15:01
让他一小段一小段比如20-50行的check in，这样你的comments可能就不算多了。

他就喜欢一大段代码，一次提交，特别长……说一下下次就不叫你了……

lxxxxxxx 发表于 2022-04-20 08:40:13
我们组一般都是service owner一定需要被included 除非是ta没那么多时间处理不过来 而且大家都一般很配合也想找个权威的人double check啊 对方如果都搞出很多问题了还不涨教训

加了team，但是会单独加另外一两个看代码比较松的人。每次都去主动看，别人还不加，我都尴尬了，也不想看了。

匿名用户 发表于 2022-04-20 08:39:58
这个就是度的把握问题，有些时候，如果不影响功能又没有bug，我会放的。

哎，就是影响啊。

匿名者 发表于 2022-4-20 11:26
哎，就是影响啊。

影响一定不让过。
有一定的坚持是对的，至于不找你review，如果是你在维护的系统相关，你可以要求作为must reviewer

匿名用户 发表于 2022-04-20 09:24:26
受教，也许是个值得尝试的方法！我就是觉得别人故意不叫我，明明知道我是service owner，而且之前的代码都我写的，我还去主动1:1会不会有点奇怪？

不会的！逐渐变成senior都要慢慢学会develop junior members。工作里沟通最重要。

lxxxxxxx 发表于 2022-4-20 10:29
。。。。。。。。。。。。。。。。。 如果是我自己代码写得烂别人帮我指出了我为什么不爽？😂

自己代码写得烂别人帮我指出，no
写的没什么大问题，别人鸡蛋里挑骨头，yes