关于团队协作的一个小问题

[zhangolve]

这两天遇到一个问题，想和大家分享一下，也不知道你们有没有遇到过。

事情是这样的，我们团队有几个前端同事，我们开发会通过提 pull request 的方式进行 review，前两天我某个同事提交代码，对之前我的一些代码进行了重构，她还特意留言告诉我让我 review，可是我当时外出了，等到回来看到消息，发现代码已经被其他同事 merge 了。我自己再看，却发现她这样的重构方式，会引起两个新的 bug 。

后来，我就把这件事情，告诉了她。结果她却告诉我说，第一个 bug，没有太大关系，可以理解为是一种用户体验。第二个 bug，可以改掉，给我说了一种解决办法，告诉我让我可以按照这个思路来改。

我的天，我看了就很不理解。因为我从业也三年了，以往在解决问题的时候，如果不是长时间的 bug，都是谁制造的问题谁去解决，因为是你制造的嘛。结果她却要我来做。。。

我也不太好直接拒绝他，因为我到这家公司不到一年，而那位同事在这个公司少说也有三年了，业务经验都比较丰富，他还算是前端组的组长，虽然没有正式封号。

所以，我想知道你们有没有遇到过此类问题，又是怎样解决的。以及他这样的作法是常见的吗。

[lhx2008]

如果是比自己级别高的话，提醒一下就行，改不改他的事。同级的就告诉他叫他自己改，他要是不改你也背不上这口锅。

[zhangolve]

@lhx2008 我们职务是平级的，都是组员。但是他的 title 比我高，他是 senior engineer，我是普通 engineer 。

[susecjh]

同感，我这边也是一个女的，来的时间久一点，自己专门写 bug，但是不改，有时间看小说也不改，就专门让别人改，但是我一直不管她，想怎么喊怎么喊，没空。

[qiaobeier]

自己拉的屎，还让别人擦屁股。

[liujavamail]

她和他，是同一个人吗

[ppd]

你们这还可以同级别的同事 merge ？不应该是更高级的 approve 之后才可以 merge 吗？

[dbskcnc]

issue 提上去，让测试重现，然后走正常流程
这个事情基本跟你没有关系啊，重构代码不是你写的，review/merge 也不是你做的

[coderluan]

”成年人只讲利益，小孩子才讲对错。“首先，这二逼话是错的。但是对于职场上发生的小概率事件，在利益没实际受损的情况下，真心没必要纠结对错浪费时间感情。楼主工作一年才遇见一次，既然当时没好意思拒绝，那就改一次呗。

[zhangolve]

@liujavamail 是同一个。

[slimyy]

he or she 麻烦分一下，看着不舒服。

[Chenamy2017]

是她你就改下，提醒她一下。
是他你就不改，提醒他一下。

[shenjixiang]

不管是男是女，告诉那个人就好了，无论如何不要去改别人的 bug

[wangyzj]

#1 和 #2 都是飞屋环游记

[mazyi]

当场说，你改出来的 bug 为啥我改？