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

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

  •  
  •   northisland · 277 天前 · 2088 次点击
    这是一个创建于 277 天前的主题,其中的信息可能已经有所发展或是发生改变。

    哎,我都无语了,

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

    我该怎么办

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

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

    哪个公司每次 commit 都要 review ?
        10
    HanSonJ   277 天前   ♥ 4
    @northisland #9 很多公司
        11
    des   277 天前 via Android
    @northisland 在自己分支上开发啊,不用每个 commit 都合并啊
        12
    raphaelsoul   277 天前
    <del>很正常阿 谁规定只有写好的代码能提交上去的?</del>

    emm 看了眼评论 svn 抱歉打扰了
        13
    zachguo   277 天前   ♥ 1
    测试里加一项:验证 demo 能跑才能 merge,不然过不了 CI 打回去。
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2278 人在线   最高记录 4019   ·  
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.1 · 23ms · UTC 11:40 · PVG 19:40 · LAX 03:40 · JFK 06:40
    ♥ Do have faith in what you're doing.
    沪ICP备16043287号-1