情绪问题还是得情感解决：最近组内冲突的感悟

我们组比较卷，可能我是最佛系的，不过没办法，压力太大，经常还是被卷进去
不过这次说的是我们组的两个最卷的同事
一个女同事，叫她咖啡吧
一个男同事，叫他牛仔吧
咖啡妹是最早的成员，比我还早，系统底层的很多东西都是她做的。一定程度上来讲，没有她就不会有现在这个项目。
她水平中上，但是特别卖力
但是她有一些缺点
1. 超敏感，轻度迫害幻想
2. 特aggressive/ambitious，特别在意credit
3. 容易情绪失控
说说她的几个事情吧
1. 一次她的一个PR被另一个组员（叫他新兵蛋子吧）reject了，原因是注释没有更新。她就说，我们的惯例是，如果是这种小问题，你可以先approve，然后提醒我修改，这样我就不用要求你再次approve。她说的其实没错。新兵蛋子很识趣啊，马上就说sorry，小弟新来的不懂规矩。本来事情就结束了。
结果另一个组员过来插了一句嘴，叫他ABC吧。ABC说，从我个人的教训来说，我倒是希望别人reject我，这样我就不会忘记修改了。说的也没错，关键人家说的是对自己严格。好了，咖啡妹不知道怎么就毛了，觉得ABC是在找事，说什么这个规则不好啊，影响效率，万一你下班了不回消息怎么办，我还着急merge代码啊，我可是对我的效率很serious的。然后ABC也没看明白状况，非得在哪里争来争去。最后把我硬拉进了一个会，我听了一会儿听明白了，两个人明明说的是两回事情嘛，真不知道有什么好吵的。ABC没眼色，咖啡妹太自我。最后我帮ABC打圆场，跟咖啡妹解释说，ABC只是想对自己严格而已，你说的惯例没问题，都散了吧。

太晚，明天接着写

concessions 发表于 2021-10-02 03:07:29
看懂了 咖啡+新兵蛋子+ABC+楼主 的故事
但是不太懂为什么一开始要介绍 牛仔，我还以为他是男主角呢

牛仔一开始没出现是因为牛仔很忙

看懂了 咖啡+新兵蛋子+ABC+楼主 的故事
但是不太懂为什么一开始要介绍 牛仔，我还以为他是男主角呢

wdyygmj 发表于 2021-10-02 09:50:22
继续写第二件事
这回牛仔出场了
牛仔有一个PR，咖啡妹根据上面的惯例approve了，留下一条提醒，一千行的代码，除了本地测试，别忘了去设备上测试一下再merge

咖啡遵守测试流程，却被说“不专业”
两个人都较真，楼主却觉得是咖啡太固执。
楼主是不是对这位咖啡成见已久lol

题外话一句，很多时候一段浅显易懂的代码都非要辛辛苦苦写unit test还要做manual test，这是行规吧。很多人因此在进prod之前及时抓住了连reviwer都没抓住的bug。从专业的角度来看，不觉得咖啡在这一点坚持自己的看法有错，牛仔不服也应该提前商讨告知 而不是默默上线。

我猜牛仔是之后的故事主角……？

🤣话说ABC这句话说得也有点轻微奇怪，但是看起来咖啡妹子是真的非常在乎/也很担心被reject，很怕临门一脚政策忽然就改了…

不对啊。如果Approve完了后，在修改后PR， 之前的Approve应该都失效了，要全部从新Approve的。

我最近也碰到这种组内冲突的，tech lead是个技术大佬，但脾气不好，各种怼我们组里的组员，我和组里另外一个senior一直在当老好人…其实大家都是打工人，不知道为什么带这么大情绪…

搁这当贴吧连载？

lilianleo 发表于 2021-10-02 07:19:54
不对啊。如果Approve完了后，在修改后PR， 之前的Approve应该都失效了，要全部从新Approve的。

我们code review用的不是github，approve之后随便怎么改之前的approve都会保留，楼主公司估计也是用类似的系统吧

lizzielee 发表于 2021-10-02 08:18:59
我们code review用的不是github，approve之后随便怎么改之前的approve都会保留，楼主公司估计也是用类似的系统吧

听着像mercurial

abc打边鼓确实没必要，尤其是在其他人情绪不好的时候 lol

继续写第二件事
这回牛仔出场了
牛仔有一个PR，咖啡妹根据上面的惯例approve了，留下一条提醒，一千行的代码，除了本地测试，别忘了去设备上测试一下再merge
结果第二天我突然看到一个@我的群消息，咱们组现在可以不测试就merge代码了么？牛仔你骗我说会去设备上测试的，早知道我就block你了

我一脸懵逼，心说又来事了啊
结果是因为她发现牛仔没理她，PR直接就merge了

然后说牛仔
牛仔这个人水平是最好的，也非常卖力，效率全组最高，可能是最差的人的两倍
他的唯一缺点是说话太直，不太注意别人的感受
比如组会上他会直接说xxx你说的不对，应该是怎么样，或者xxx你这样做不好，应该如何如何。
事实层面上来说，他基本上都是对的，但是语言上听起来就有点harsh。他还挺擅长argue，当然是理性argue那种风格
他对谁都这样，包括对我，所以我倒是相信他是真心的为项目考虑，只是个人风格不同罢了

好了，他们两个在一起就热闹了
牛仔说，别看一千行，都是boilerplate。所有的代码在本地都测试了，而且本地测试工具完全可以cover各种场景，没必要上设备测试，因为这个任务很紧急，设备测试来回得多花一天

咖啡当然不接受，她说你这是只为了自己的progress不遵守coding standard，太unprofessional

牛仔刚开始还比较克制，他说对不起，我没骗你，我只是觉得确实没必要

咖啡不依不饶，继续bb

牛仔最后急了，我这都已经merge了，你现在到底想要我怎么办呢？

咖啡说，我就想要你去把测试跑一遍

牛仔说，好吧，我去测，希望能赶在release之前。

完了咖啡拉了一个会，非得让我站边

我说，两位都是我组的得力干将，都非常的professional，而且我相信你们都是为了项目好。今天这个事情呢，在技术上确实比较模糊。原则上如果reviewer有concern，她有权力block PR。另外，出于代码质量考虑，不确定的话就多测试一下总归没错。那既然牛仔道歉了，也把测试file了，咱们就重申一下policy，大家同意的话以后就不会有冲突了。

我觉得咖啡明显很冲动，在会上还是不要讨论她的情绪问题了，先这么着吧。

事情暂时平息
显然两个人的情绪并没有平息

1 on 1的时候，牛仔就说，我觉得咖啡是在针对我个人，她太不专业。我最后其实就是屈服了。我说，谢谢你的顾全大局，虽然技术上我倾向于保险一点多跑一遍测试没错。但是她确实情绪过火了。她确实有不少情绪控制的问题，我知道她也在努力，人都不一样，咱们尽量focus在如何合作上。

后来我从我老板那里得知，咖啡还去我老板那里抱怨我没有及时发声挺她

好了，不管怎么样，这件事情暂时结束了