这是一个创建于 1676 天前的主题，其中的信息可能已经有所发展或是发生改变。
这两天遇到一个问题，想和大家分享一下，也不知道你们有没有遇到过。
事情是这样的，我们团队有几个前端同事，我们开发会通过提 pull request 的方式进行 review，前两天我某个同事提交代码，对之前我的一些代码进行了重构，她还特意留言告诉我让我 review，可是我当时外出了，等到回来看到消息，发现代码已经被其他同事 merge 了。我自己再看，却发现她这样的重构方式，会引起两个新的 bug 。
后来，我就把这件事情，告诉了她。结果她却告诉我说，第一个 bug，没有太大关系，可以理解为是一种用户体验。第二个 bug，可以改掉，给我说了一种解决办法，告诉我让我可以按照这个思路来改。
我的天，我看了就很不理解。因为我从业也三年了，以往在解决问题的时候，如果不是长时间的 bug，都是谁制造的问题谁去解决，因为是你制造的嘛。结果她却要我来做。。。
我也不太好直接拒绝他，因为我到这家公司不到一年，而那位同事在这个公司少说也有三年了，业务经验都比较丰富，他还算是前端组的组长，虽然没有正式封号。
所以，我想知道你们有没有遇到过此类问题，又是怎样解决的。以及他这样的作法是常见的吗。
 |
1
lhx2008 2020-04-22 09:15:09 +08:00 via Android
如果是比自己级别高的话，提醒一下就行，改不改他的事。同级的就告诉他叫他自己改，他要是不改你也背不上这口锅。
|
 |
2
zhangolve OP @lhx2008 我们职务是平级的，都是组员。但是他的 title 比我高，他是 senior engineer，我是普通 engineer 。
|
 |
3
susecjh 2020-04-22 09:34:08 +08:00 via Android
同感，我这边也是一个女的，来的时间久一点，自己专门写 bug，但是不改，有时间看小说也不改，就专门让别人改，但是我一直不管她，想怎么喊怎么喊，没空。
|
 |
4
qiaobeier 2020-04-22 09:48:11 +08:00
自己拉的屎，还让别人擦屁股。
|
 |
5
liujavamail 2020-04-22 09:52:12 +08:00
她和他，是同一个人吗
|
 |
6
ppd 2020-04-22 09:56:07 +08:00
你们这还可以同级别的同事 merge ？不应该是更高级的 approve 之后才可以 merge 吗？
|
 |
7
dbskcnc 2020-04-22 10:14:57 +08:00
issue 提上去，让测试重现，然后走正常流程
这个事情基本跟你没有关系啊，重构代码不是你写的，review/merge 也不是你做的 |
 |
8
coderluan 2020-04-22 10:24:47 +08:00
”成年人只讲利益，小孩子才讲对错。“首先，这二逼话是错的。但是对于职场上发生的小概率事件，在利益没实际受损的情况下，真心没必要纠结对错浪费时间感情。楼主工作一年才遇见一次，既然当时没好意思拒绝，那就改一次呗。
|
|
9
zhangolve OP @liujavamail 是同一个。
|
 |
10
slimyy 2020-04-22 10:42:33 +08:00  1
he or she 麻烦分一下，看着不舒服。
|
 |
11
Chenamy2017 2020-04-22 10:47:03 +08:00
是她你就改下，提醒她一下。
是他你就不改，提醒他一下。 |
 |
12
shenjixiang 2020-04-22 10:58:01 +08:00
不管是男是女，告诉那个人就好了，无论如何不要去改别人的 bug
|
 |
13
wangyzj 2020-04-22 12:43:23 +08:00
#1 和 #2 都是飞屋环游记
|
 |
14
mazyi 2020-04-22 15:13:03 +08:00 via iPhone
当场说，你改出来的 bug 为啥我改？
|
Select Language