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

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

哈哈，我司的系统一直有这个漏洞，具体原因我没研究。不过大部分时候大家都比较自觉，approve之后如果有重要改动，还是会要求再次review

两个人各执己见，楼主却用了两种倾向的词，咖啡用贬义词，自我，abc用“亲切包容的”中性词，没眼色。咖啡在会上被打断被abc较真，结果楼主却觉得是咖啡固执。emm可以思考一下自己的判断体系是否不公平hh
提供一个清奇的视角lol

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

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

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

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

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

插个眼 紫薯

烤馒头 发表于 2021-10-02 13:28:01
牛仔一开始没出现是因为牛仔很忙

xswl今日最佳，脱口秀新星

咖啡看起来没什么大错啊

码农是真的无聊啊 还不够无聊的吗 一点代码review来搞出这么多p事来

烤馒头 发表于 2021-10-2 13:28
牛仔一开始没出现是因为牛仔很忙

我想抖个机灵米都加到你那里去了，分我点呗