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

服了,一个同事写的 demo,每次他 commit 后,我不经过曲折的 demo 代码修改,肯定是跑不通的。

  •  
  •   northisland · 2018-03-13 19:41:54 +08:00 · 3019 次点击
    这是一个创建于 2206 天前的主题,其中的信息可能已经有所发展或是发生改变。

    哎,我都无语了,

    天知道他是怎么确定这次 commit 是正确的。。。

    我该怎么办

    13 条回复    2018-03-14 10:20:16 +08:00
    kosilence
        1
    kosilence  
       2018-03-13 19:46:21 +08:00
    Need code review.
    kosilence
        2
    kosilence  
       2018-03-13 19:46:48 +08:00   ❤️ 2
    And precommit unit test.
    fanqianger
        3
    fanqianger  
       2018-03-13 19:48:26 +08:00
    你需要单元测试吧,一般提交 review 后都会自动跑单元测试,如果没有通过,是不会有人 review 你的代码的
    northisland
        4
    northisland  
    OP
       2018-03-13 19:51:50 +08:00
    想太多了,日常开发,公用一个 SVN (我也觉得 git 好,只是公司用的是这个)
    scnace
        5
    scnace  
       2018-03-13 19:52:37 +08:00 via Android   ❤️ 1
    svn 好像也可以 hook
    zjsxwc
        6
    zjsxwc  
       2018-03-13 19:56:26 +08:00 via Android
    连 git 都不愿意切换的项目,让他们自己玩去吧
    shynome
        7
    shynome  
       2018-03-13 19:59:14 +08:00 via Android   ❤️ 1
    永远不动别人的代码。
    你改好了,他又会改回去的,得一起找出问题才行
    3a3Mp112
        8
    3a3Mp112  
       2018-03-13 20:00:02 +08:00   ❤️ 1
    没人 review 的代码是怎么 commit 成功的?
    northisland
        9
    northisland  
    OP
       2018-03-13 20:10:26 +08:00
    @3a3Mp112 我不是他上司,也不是他导师……

    我们出大版本,是要 review,jekins 的,

    哪个公司每次 commit 都要 review ?
    HanSonJ
        10
    HanSonJ  
       2018-03-13 20:11:38 +08:00   ❤️ 4
    @northisland #9 很多公司
    des
        11
    des  
       2018-03-13 20:19:04 +08:00 via Android
    @northisland 在自己分支上开发啊,不用每个 commit 都合并啊
    raphaelsoul
        12
    raphaelsoul  
       2018-03-13 20:22:37 +08:00
    <del>很正常阿 谁规定只有写好的代码能提交上去的?</del>

    emm 看了眼评论 svn 抱歉打扰了
    zachguo
        13
    zachguo  
       2018-03-14 10:20:16 +08:00   ❤️ 1
    测试里加一项:验证 demo 能跑才能 merge,不然过不了 CI 打回去。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   4007 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 10:20 · PVG 18:20 · LAX 03:20 · JFK 06:20
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.