V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
zhangolve
V2EX  ›  程序员

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

  •  
  •   zhangolve · 2020-04-22 09:04:52 +08:00 · 2137 次点击
    这是一个创建于 1436 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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

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

    14 条回复    2020-04-22 15:13:03 +08:00
    lhx2008
        1
    lhx2008  
       2020-04-22 09:15:09 +08:00 via Android
    如果是比自己级别高的话,提醒一下就行,改不改他的事。同级的就告诉他叫他自己改,他要是不改你也背不上这口锅。
    zhangolve
        2
    zhangolve  
    OP
       2020-04-22 09:17:55 +08:00
    @lhx2008 我们职务是平级的,都是组员。但是他的 title 比我高,他是 senior engineer,我是普通 engineer 。
    susecjh
        3
    susecjh  
       2020-04-22 09:34:08 +08:00 via Android
    同感,我这边也是一个女的,来的时间久一点,自己专门写 bug,但是不改,有时间看小说也不改,就专门让别人改,但是我一直不管她,想怎么喊怎么喊,没空。
    qiaobeier
        4
    qiaobeier  
       2020-04-22 09:48:11 +08:00
    自己拉的屎,还让别人擦屁股。
    liujavamail
        5
    liujavamail  
       2020-04-22 09:52:12 +08:00
    她和他,是同一个人吗
    ppd
        6
    ppd  
       2020-04-22 09:56:07 +08:00
    你们这还可以同级别的同事 merge ?不应该是更高级的 approve 之后才可以 merge 吗?
    dbskcnc
        7
    dbskcnc  
       2020-04-22 10:14:57 +08:00
    issue 提上去,让测试重现,然后走正常流程
    这个事情基本跟你没有关系啊,重构代码不是你写的,review/merge 也不是你做的
    coderluan
        8
    coderluan  
       2020-04-22 10:24:47 +08:00
    ”成年人只讲利益,小孩子才讲对错。“首先,这二逼话是错的。但是对于职场上发生的小概率事件,在利益没实际受损的情况下,真心没必要纠结对错浪费时间感情。楼主工作一年才遇见一次,既然当时没好意思拒绝,那就改一次呗。
    zhangolve
        9
    zhangolve  
    OP
       2020-04-22 10:27:47 +08:00
    @liujavamail 是同一个。
    slimyy
        10
    slimyy  
       2020-04-22 10:42:33 +08:00   ❤️ 1
    he or she 麻烦分一下,看着不舒服。
    Chenamy2017
        11
    Chenamy2017  
       2020-04-22 10:47:03 +08:00
    是她你就改下,提醒她一下。
    是他你就不改,提醒他一下。
    shenjixiang
        12
    shenjixiang  
       2020-04-22 10:58:01 +08:00
    不管是男是女,告诉那个人就好了,无论如何不要去改别人的 bug
    wangyzj
        13
    wangyzj  
       2020-04-22 12:43:23 +08:00
    #1 和 #2 都是飞屋环游记
    mazyi
        14
    mazyi  
       2020-04-22 15:13:03 +08:00 via iPhone
    当场说,你改出来的 bug 为啥我改?
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3534 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 10:46 · PVG 18:46 · LAX 03:46 · JFK 06:46
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.