如何优雅的给自封lead的同事写review

新人进组第一个full cycle，做了一个完整的feature。由于一开始的mentor离开我还不到6个月，组里老哥A接替之后开始weekly1:1。今天review看到组里老哥直接写“reviewed and steered”这个work。实际上老哥确实是reviewed了基本所有diff（除了TL以外他对这个比较熟，TL太忙了），但是老在有的没得一些地方卡我（有时候不得不去TL那要帮忙review，A哥实在是看的又慢，提的东西又有点不现实（有时候墨迹一两回合说你这设计的不好，暗示重构。有些东西有些道理，但真的有点不切实际，加个feature上来就重构。。感情干活的不是你。最后TL支持先不要搞这么大改动，以后慢慢来）），最终还是按照我的思路land了。另外我俩同级别，A比我早来快一年。

问题来了：他自评中这个“steered”看的我就有点。。。撕是不合适，但是怎么能看似夸奖但是其实说“你ya steer了个xx”的review这个project，正在斟酌语句，不知道有没有过来人支几招，小弟提前谢过了

补充：他还说made most decision making in regards to evolution of xxx。。。🤮

这种明显抢你credit夸大自己的impact，口头或email把你的feedback告诉老板，明确说 as project owner, I disagree with his self assessment on his impact.
写他的review就提一下he actively involved and offered suggestions什么的，也可以说among the suggestions, xxx was considered helpful and implemented by the project owner. 然后给自己写一个这个project owner的self review就行了.

你以后要是review了他的work的话 你也说review and steer了他的project呗

感谢involve.

同时质疑他来steered是不是对组里定下的ownership/boundary不满意, 还是对楼主的drive speed/process有疑问(有疑问是不是应该先callout)? 是不是缺乏grow others 的mind set?

楼主信息提供的太少，不好帮忙啊。

同一个级别的人，水平也是千差万别，刚升到这个级别的人被在这个级别待了两三年等着promo的人带一下也是理所当然的。

楼主了解老板的想法吗？老板对老哥有什么期待？这种辅导关系是不是老板直接安排用来培养老哥的？我有个朋友（真的不是我自己哦，哈哈），进组没多久就跟组里老人不对付，后来发现老人很快升职做他老板了，于是黯然离职了。

nbspm 发表于 2021-8-31 14:54
楼主信息提供的太少，不好帮忙啊。

同一个级别的人，水平也是千差万别，刚升到这个级别的人被在这个级别待 ...

我已经submit了所以whatever了，直接在manager能看到的那一栏里说这一条描述不对（先说感谢贡献xxx，但是我感觉collaborate rather than xxx，说的挺直白但没硬刚）。

A哥也没比我来的早多少，当时带我基本是tl忙不过来，本来的mentor走人了，还有一个sr休产假。a哥讲道理也不像搞政治斗争的人，我也不知道为啥搞这么一出。可能是a哥本来最重要的项目卡在legal没什么进展所以想整点活儿，问题是直接claim别人的活儿是自己lead的就有点过了。我基本确定manager是把这个活儿安排给我作为第一个full cycle的主打项目之一的（sync 过），而且 /这个活儿不难，根本用不到两个人，而且没什么值得lead的/，他从mentoring的角度出发我也就忍了。要是没这一出我都打算闭眼给他打exceed了，好吧，那我只能说“a哥你项目卡着launch不了但是我觉得你依然solid，非常meet expectation”

我觉得lz有点心态不端正了 应该在狗家吧 狗家eng review比较挑剔很正常啊
人家花时间在review上 跟你讨论具体该怎么做 有什么更好的方案 就是在steer the work啊  🤔 你自己也说了有些地方的改进你觉得不认同 但也有些是应该的
问题是这个Project 他又不写 并不能算是合作
这么aggressive在review这么说 老板不会觉得你特别好说话

zzyjason 发表于 2021-8-31 17:14
我觉得lz有点心态不端正了 应该在狗家吧 狗家eng review比较挑剔很正常啊
人家花时间在review上 跟你讨论 ...

感谢建议。。我也就是发帖表现的有点aggressive，主要确实是不是很爽需要发泄下垃圾情绪，但是review
留言/评语里还是有分寸的。层主你看我补充的那个quote，我觉得挺明显a同事是把自己picture成了owner/leader。个人认为这种事情不能什么都不说，除非是manager的授意，不然manager是不可能明察秋毫的，然后就很有可能留下入职快一年还需要跟着同级别的人干杂活的印象，如果恰好这个活儿也不难，那对于你build trust来说就非常的不利了（new grad，或者高级带低级不在讨论范围）。你换一种场景想想，你写了个design doc，加了几个组员为reviewer，如果最后除了tech lead以外的人跑出来说自己steer the work，made most decision etc，理由是花时间review了你的doc和你的代码，你同意吗？

99oamiad 发表于 2021-09-01 09:55:14
感谢建议。。我也就是发帖表现的有点aggressive，主要确实是不是很爽需要发泄下垃圾情绪，但是review
留言/评语里还是有分寸的。层主你看我补充的那个quote，我觉得挺明显a同事是把自己p

hmmm 我觉得他可能应该说是contributed to the evolution of the project 可能好一些

要是我组里有人这么做了 而且情况跟你一样 在代码上有很大分歧 我觉得ta这么写也没问题的

我这个cycle的review 有你这样的情况  就我自己设计了这个feature 但是和另外一个组员讨论了很久
我自己perf就会写上跟他一起设计出来的 然后那个组员跟我说没必要把我写上去  你就写自己independently design 和implemented the feature就行 😂

我觉得你要是完全就自己写的 自己设计的 别人也就看了看你的代码没有给设计上的建议 你这么做没问题 但作为新人 你得到了帮助 就写下来 哪怕多给点credit出来都没问题我觉得