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

请问各位大佬，作为一个刚入职场的新人，看公司文件上对我这个级别的期待中是包含贡献 code review的，但是组里的其他人几乎都比我呆的时间久，或者level更高。这种情况下，我还可以给他们 code review吗？

其实对于比较senior的人，楼主能提出的也是无伤大雅，optional的comments。当然我给comments的语气是很客气的。

如果让我选，我是能不写就不写，但又怕performance review时候会被挑这个方面没有贡献。。大家有什么建议吗? 我可以给他们CR吗？还是我都点approve也算贡献了😅

或许我可以咨询下mentor，看看组里大家的习惯是怎么样？

补充内容 (2023-05-08 00:03 +8:00):
谢谢大家回复，受益良多。补充一下，楼主指的情况是自己主动去找PR来 review。

我们组一般是有一些需要review的PR放在那里，谁想review，就去挑。所以这种时候楼主想要更多参与CR，也来问大家是否是常规操作，有哪些需要注意的。

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

当然可以。但是尽量给些有实际意义的comment。给sr挑一堆语法或者命名之类的。人家就会不乐意了

正常review 没关系
要是senior有不满    那是他人有问题

反过来说，作为senior，如果我让junior或者新同事来review我的代码，一般出于这几点考虑：
- 希望他们能更好的了解我改动的代码/系统，通过review我的MR来更好地理解系统是怎么运作的，然后可以以此为参照做他们的活，follow the practice。
- 希望他们指出代码中不太好理解的部分，我可以改动一下或者加上更好的文档/注释。毕竟我不可能一辈子维护那些代码，如果有人看不懂，那我有必要改进readability。
- 新人也更容易指出系统里老人觉得理所应当实则被忽视的问题。

至少code readability上可以谈谈 哪里看不懂要求clarify一下

为什么不review，这个跟级别没有任何关系，这个是你工作的一部分。

至于么，该怎么review就怎么review，说的有理就行，以后review code多了去了，难道还要先看下对方什么级别？Jeff Dean想check in点code也需要找owner review阿，级别不都低了Ｎ级

可以的，review的过程也是互相熟悉的过程

如果review做的好，senior也能看到你的能力，能力有两方面，一部分是技术能力，另一部分是态度和为人处事，两方面都很重要。

我们组3YOE+的人写的code都没人给comment，秒approve
美其名曰“already earned trust”

难道这不是问问题和学习的好机会么

换个角度如果是我的PR，然后看到别人说些无伤大雅的optional comments其实会有点头疼，不改的话会担心让你觉得我是在无视你，改的话又觉得浪费时间没啥意思。
所以我自己给别人的comments可有可无的我都不会写，要写的就是真的是哪儿有问题的。