不认识的同事来要peer feedback咋办？

lz新人，和这同事不认识，或者说根本不熟吧（公共channel交流不超过5句message）。是同一个department的。


peer feedback request上说，请说说XX package如何帮助你写automated test的。我才明白我之前做的一个feature，自己加了一两个automated tests，用到了他的做的package。他的package是把一些production api做到了test api里面，然后这个package被util和base用了，所以我的test也就用了。话说我本来也就是在一个已有的suite里面加了一两个test case。大家可能明白到此的逻辑。

此同事是sdet，主要就是写automated test和维护pipeline，所以我觉得这个package对他很重要。但是我觉得peer feedback是写给共事过的同事写的吧，比如两人在这个项目上合作过（不管是写code，写doc，或者很多communication，等等），是针对人写的。像这种用到package的事，自己在review的时候提一下，自己的package被AA， BB，CC这些用到了，不就行了，而且git都有，直接明了。

lz不知道该不该写，该如何写。自己新人，不知道此地的要求，万一org的风气就是这样？（其他同事也有request，一个是同组的，一个是隔壁组，两位都是合作项目，所以我觉着这peer review/feedback 才正常吧。自己同一个组，但目前没有合作过的都没request，还别提同组的code review的交流都比这位package同事多得多了）。写了，下次又来找我写这些margin的use case咋办？毕竟peer feedback写一两句打发了也不好吧。

真是左右为难。万能地里能支招吗？

我建议写。就把你那个test吹一下，因为有了他这个package，你的test才容易做，而且能automate，省了很多时间。这样就是互相给credit，岂不完美。以后或许你也需要找他要feedback，他肯定也会把你吹一番。

如果是我，我肯定会写。在不熟悉的情况下，能写出漂亮的feedback也是工作能力之一。
公司里面人那么多，你不可能都熟悉。

我以前遇到过，不给我写feedback的人。其实也没什么。但是我心里肯定不开心一辈子。当然，我这样想是不对的，但是人就是这样，会有永久记忆。
如果以后我有机会发达了，或者那个人要求我做事情，我肯定会长个心眼，知道这人是死脑筋，不好对付。因为你以前没给我写feedfack，以后找我做事，我最多只会公事公办，而且我很可能还会带放大镜，放大10万倍，去找茬。

当然，这些都是我心里的小恶魔。实际上我不会这么做，但是心里总是不开心。

如果你不给人写feedback，那人就会这样思考。

另外，我还遇到过，连合作都没合作过人的人，找我写feedback。我当时现场学习，搜他的代码和文档，根据他的寥寥几句总结，看看他做了什么，有什么闪光点。然后给他好好吹了一番，写了一份金光灿灿的feedback。

以前在这种有写peer review的公司, 每个人要找5-7个peer, 写的人头疼, 找的人也头疼
所以特别不熟的, 可以不写
用过几个package的, 你就写这个package到底有没有帮上你的忙
只有比较熟的, 再按需求一条条地写

想想申绿卡的时候，也是找八杆子打不着的人写推荐信。先是请那些引用过我的文章的老师，一些被默拒了之后，甚至找我引用的文章的作者，卑微地请人家读一下我的文章，再帮忙推荐信 ～～～～～～

楼主如果有时间的话，还是写一下吧。Review长短是其次的， 关键你是否愿意帮忙

1-1的时候问问你的manager，了解一下这方面culture。另外，你用了他做的东西是他的consumer，找你要feedback也很正常。有建议提建议，没建议说很好。如果feedback能被他老板看到，最好写得客气点，建设性意见，对事不对人。

匿名用户 发表于 2022-9-7 08:59
1-1的时候问问你的manager，了解一下这方面culture。另外，你用了他做的东西是他的consumer，找你要feedbac ...

谢谢。主要是之前我写peer review，都是会很认真找case，一条条写上去。所以这个package同事找我写，我还真觉得没有一点可写的。那个package是suite用的，所以我对它也不了解。

就如实写吧 我可能会帮忙吹几句
形成互助的文化对大家都有好处

直接说不熟，不写。learn to say no.

zoevelynne 发表于 2022-9-7 09:53
以前在这种有写peer review的公司, 每个人要找5-7个peer, 写的人头疼, 找的人也头疼
所以特别不熟的, 可以 ...

谢谢你的意见。这个确实不熟，而且一个package，还是在suite level。从他的角度，我是借他光了；从我的角度，我就加了一个case，我都没体验到package，确实不知道咋写。要不只有吹一下。

rocket_science 发表于 2022-9-7 12:59
直接说不熟，不写。learn to say no.

我也这样想过：不熟，没合作过不知道怎么写。给package feedback，应该没问题；给人feedback，我写不出

匿名用户 发表于 2022-9-7 15:44
谢谢你的意见。这个确实不熟，而且一个package，还是在suite level。从他的角度，我是借他光了；从我的角 ...

那就不一定非得要写吧。。。