作为新人，可以给比我senior的人Code Review吗？

吾爱吾师，吾更爱真理

austurela 发表于 2023-5-7 02:23
要我说就一条
senior的人想让你review吗？如果人家根本不需要你review，别去添乱
如果人家找你review，那 ...

可能不会主动找我review吧，毕竟找我不如找其他senior review更有说服力吧。但那如果一直没人找我review，我最后一点CR contribution都没有怎么办？😂 而且我看我们组大家也不会说经常指定谁review，一般都是谁想review就自己挑PR吧

jhstjh 发表于 2023-5-7 03:52
反过来说，作为senior，如果我让junior或者新同事来review我的代码，一般出于这几点考虑：
- 希望他们能更 ...

谢谢回复。那你们组的新人一般都是被assign了才会去review senior的代码吗？还是你作为senior也不介意他们可以自己主动去review？

谢谢大家回复，受益良多。补充一下，楼主指的情况是自己主动去找PR来 review。我们组一般是有一些需要review的PR放在那里，谁想review，就去挑。所以这种时候楼主想要更多参与CR，也来问大家是否是常规操作，有哪些需要注意的。

如果是别人指定找我review，那我是不会拒绝的，虽然我感觉这种情况会很少。

当然可以，review是一个互相学习的过程。能给出有价值的意见，谁都喜欢你

spitz6860 发表于 2023-05-06 17:00:40
换个角度如果是我的PR，然后看到别人说些无伤大雅的optional comments其实会有点头疼，不改的话会担心让你觉得我是在无视你，改的话又觉得浪费时间没啥意思。
所以我自己给别人的commen

的确是，最烦挑一些nits的。

营养贴，引起有意义的讨论！

匿名用户 发表于 2023-05-07 08:58:39
谢谢大家回复，受益良多。补充一下，楼主指的情况是自己主动去找PR来 review。我们组一般是有一些需要review的PR放在那里，谁想review，就去挑。所以这种时候楼主想要更多参与CR，也来问大

你们组很有意思诶这个做法
我能想到的点有
看到那种重要的小改动快速去approve，我真的超喜欢在这种事情上帮我unblock的人。
看到自己不会的东西留个TIL。这样可以显示你在主动学习，author也有成就感。
给别人留comments尽量有依据。让自己的每个意见都显得质量高，奠定自己在组里的技术地位。

匿名用户 发表于 2023-5-7 17:43
你们组很有意思诶这个做法
我能想到的点有
看到那种重要的小改动快速去approve，我真的超喜欢在这种事情上 ...

谢谢你的建议，看来有很多组是author指定人来review对吗？那好奇会不会有人很少被人指定来review，结果他的CR数量就很少，会不会影响以后升职？

TIL这个建议很好！我看了很多PR其实都没发现什么需要改动的，而且我目前更多的是处于在别人的PR里学习的状态，所以很多时候看完了这个PR我就默默关了，别人也不知道我看过了。

我也经常看到，有的人approve了，但又有其他人给了新的comments。

不要一开始就去看代码，先想想这个story或是bug或是feature，如果自己来做，怎么去找到相关性，怎么去找到以前类似的场景/代码，如果自己入手，怎么去实现，怎么去测试？性能怎么兼顾，需要拓展性吗？需要监测吗？

带着自己的问题去review就会比较容易去读懂代码，甚至可以去感受搜索一下代码是不是有没有照顾到的地方