V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
pink123
V2EX  ›  职场话题

如果遇到一个一周才提交一次代码的同事,该怎么办?

  •  
  •   pink123 · 2019-05-28 15:09:37 +08:00 · 7911 次点击
    这是一个创建于 2011 天前的主题,其中的信息可能已经有所发展或是发生改变。

    身为一个码农,深知代码版本控制的重要性,可就是有一些人野路子出身,可能之前在小公司一个人说了算,所以从来没有养成代码版本管理的习惯,遇到不提交代码的同事该怎么办呢?

    65 条回复    2019-07-22 16:30:51 +08:00
    yalin
        1
    yalin  
       2019-05-28 15:12:55 +08:00
    或许人家才写到一半
    402124773
        2
    402124773  
       2019-05-28 15:14:14 +08:00
    设定 gitflow。让他提交代码前自己解决冲突,哈哈。
    goreliu
        3
    goreliu  
       2019-05-28 15:17:10 +08:00 via Android   ❤️ 8
    如果他这样提交代码不影响你的工作,那对你来说不存在问题,不需要解决。

    如果已经影响了你的工作,那以出现的实际影响为由和他沟通,而不是以良好习惯为由。如果沟通不顺畅,那以实际影响为由找上级沟通。
    690690
        4
    690690  
       2019-05-28 15:39:59 +08:00
    结果导向,其它,别在乎太多
    lloovve
        5
    lloovve  
       2019-05-28 15:48:45 +08:00 via iPhone   ❤️ 4
    遇到一个有事没事就提交代码的怎么办
    spritewdx
        6
    spritewdx  
       2019-05-28 16:17:03 +08:00   ❤️ 1
    良好沟通才能解决问题,而不是埋怨瞧不起,土八路也不一定干不过正规军
    yimity
        7
    yimity  
       2019-05-28 16:21:15 +08:00
    我一周就做一个功能,你非要我天天提交么?
    shm7
        8
    shm7  
       2019-05-28 16:23:19 +08:00
    还是要多沟通。写一篇文章专门讲这个的,并发给他看。
    linxl
        9
    linxl  
       2019-05-28 16:26:22 +08:00   ❤️ 1
    我同事也这样, 沟通以后好多了, 说说弊端他就能接受, 毕竟一周一坨一个 commit 以后他自己也没发看.
    xiaoyang7545
        10
    xiaoyang7545  
       2019-05-28 17:57:58 +08:00
    不过要是写到一半的,传了别人又说有 BUG 要修复咋办。
    aleen42
        11
    aleen42  
       2019-05-28 18:52:03 +08:00
    protect 好分支,管他写一年
    MeteorCat
        12
    MeteorCat  
       2019-05-28 18:54:33 +08:00 via Android
    现在提交都要按天数了?我们都是按迭代版本提交的
    SuperMild
        13
    SuperMild  
       2019-05-28 19:06:12 +08:00
    这不是领导要负责的东西吗,领导拿着高薪,就是处理这些“人为”问题的呀。
    warcraft1236
        14
    warcraft1236  
       2019-05-28 19:06:39 +08:00
    之前一同事,每次说起来这种事都说他从不提交代码,手动狗头
    ensonmj
        15
    ensonmj  
       2019-05-28 19:37:00 +08:00
    难道不是按需求来吗?
    TuringGunner
        16
    TuringGunner  
       2019-05-28 19:40:23 +08:00 via Android
    不题交也不会影响你啊

    反正在他自己的分支上
    1OF7G
        17
    1OF7G  
       2019-05-28 19:42:20 +08:00
    有没有人解释下这样做的弊端?
    个人感觉,没啥影响啊,除了到时候一个 commit 的内容比较多。
    exonuclease
        18
    exonuclease  
       2019-05-28 19:45:15 +08:00 via iPhone
    冲突他自己解就行
    ivanchou
        19
    ivanchou  
       2019-05-28 19:46:20 +08:00 via Android
    @goreliu 必然影响啊 一大坨代码怎么 review
    tantalu
        20
    tantalu  
       2019-05-28 20:15:51 +08:00
    按照 git 提交次数统计工作量
    kiyo
        21
    kiyo  
       2019-05-28 20:38:40 +08:00
    让他拉个分支自己玩咯...
    lygmqkl
        22
    lygmqkl  
       2019-05-28 20:40:15 +08:00   ❤️ 1
    不是故意喷你, 什么叫野路子? 频繁提交就有优越感了?

    好的公司管理应该是,结果导向并且合理的拆分任务,按照你的题意,你们公司居然可以一周提交一次也不影响开发进程,只能说明你们的管理很混乱。

    综上: 代码多久提交一次并不重要(国企除外), 主要是一个固定时间范围内有多少 tasks 分配下来,并且完成度如何,同时计算下 bug 率和相关的数据就可以了。
    lygmqkl
        23
    lygmqkl  
       2019-05-28 20:40:48 +08:00
    @tantalu 那我能提交到 让老板回家卖房你信不信。。。 :)
    onehao28
        24
    onehao28  
       2019-05-28 20:55:28 +08:00 via Android
    按需求提交的路过,有时候两三周才提交一次很正常。
    winterock
        25
    winterock  
       2019-05-28 22:42:27 +08:00
    当你在一个有上千人协作的 repo 上工作,你就会知道那些没想清楚没测试完毕就往 master 上 push 的人有多“野路子”了。

    一般都是在自己的 branch 上开发,不定期 rebase 到最新的 master。
    开发完了,写非常详细的 merge request,找人 review。
    最后,merge 的时候一般还会 squash,可能十几个 commits 就成了你说的一坨一个大的 commit 了(但是这是必须的,没人想看你那些啰里八嗦的细节,实在要看的话回头翻 merge request 就好了)。
    Titzanyic
        26
    Titzanyic  
       2019-05-28 22:53:24 +08:00 via iPhone
    和出身和是否小公司没关系,主要是个人习惯。我一个人写代码,按照功能提交,创建合并分支。
    l00t
        27
    l00t  
       2019-05-28 22:57:30 +08:00
    合并到主干的话,一个功能一提交,一个需求一提交才好啊。review 当然也是有明确的需求再去 review 比较方便啊。不然一堆的修个笔误、加段代码、又把那段代码删了、想了想又加回去并改几个细节、改几个命名…… 这种提交看过去很爽吗?
    CFM880
        28
    CFM880  
       2019-05-28 23:20:32 +08:00
    重构一部分,强制用 rebase, 不准用 merge,让他自己解冲突,绝对解冲突到哭
    winterbells
        29
    winterbells  
       2019-05-28 23:43:57 +08:00
    @lloovve 这不说的就是我,commit 数量是其他人加起来两三倍。。。不过都是独立模块提交到自己分支==
    stallman
        30
    stallman  
       2019-05-28 23:59:49 +08:00
    还能怎么样,舔他啊
    taoky
        31
    taoky  
       2019-05-29 01:16:43 +08:00
    @xiaoyang7545 `git stash`
    arischow
        32
    arischow  
       2019-05-29 07:57:25 +08:00 via iPhone
    如果是按照功能开发完成再合并到主分支,有什么问题么?这样 commit 大小只影响他本人吧
    zyp0921
        33
    zyp0921  
       2019-05-29 08:48:11 +08:00
    编不过也要提交吗??? 妈耶
    tt67wq
        34
    tt67wq  
       2019-05-29 09:29:54 +08:00
    说不定在别的分支呢
    yxcoder
        35
    yxcoder  
       2019-05-29 09:34:35 +08:00
    3 天及以上起新分支
    hotwater
        36
    hotwater  
       2019-05-29 09:47:00 +08:00
    小公司野路子的码农表示被深深地伤害了。。。。
    hjq98765
        37
    hjq98765  
       2019-05-29 10:06:48 +08:00
    又不是 SVN,自己拉个分支多简单,或者要求他拉个分支
    qq976739120
        38
    qq976739120  
       2019-05-29 10:36:59 +08:00
    经历了三家公司...都是一个功能一个分支的,一个分支上好几个功能修改的话,岂不是很头疼
    sdushn
        39
    sdushn  
       2019-05-29 11:15:32 +08:00 via Android
    估计是没有提代码的习惯,解冲突多累😂而且有划水嫌疑啊,一般情况下我一天一提交就是在划水了😂😂😂
    sdushn
        40
    sdushn  
       2019-05-29 11:15:43 +08:00 via Android
    @sdushn (比如现在)
    zjyl1994
        41
    zjyl1994  
       2019-05-29 13:54:48 +08:00
    自己拉分支自己 merge 喽,到时候炸多了就知道赶快提交 merge 的重要性了
    iyaozhen
        42
    iyaozhen  
       2019-05-29 14:02:57 +08:00
    没关系呀 自己一个分支呗,最好提交的还得解决冲突

    但如果两人合作一个功能,那肯定得提高提交频率了
    iyaozhen
        43
    iyaozhen  
       2019-05-29 14:03:12 +08:00
    @iyaozhen 最好提交=最后提交
    magic3584
        44
    magic3584  
       2019-05-29 14:07:30 +08:00
    "野路子"emmm 楼主好正规军哦
    php01
        45
    php01  
       2019-05-29 14:20:10 +08:00
    原来科班出身的是这样的呀,懂了。
    Mac
        46
    Mac  
       2019-05-29 14:21:08 +08:00 via Android
    人家可以每天改一下版本号
    younfor
        47
    younfor  
       2019-05-29 15:12:20 +08:00
    鹤鹤,胖友你是没在国企呆过啊,一周只有一次代码提交权限,错过时间就没有机会了,还是集中式提交。怕了。
    mawenjie
        48
    mawenjie  
       2019-05-29 16:43:03 +08:00
    说不定别人打算把模块测试好了再提交代码,稳重一点。最可怕的是那种提交之前都不编译的
    unco020511
        49
    unco020511  
       2019-05-29 17:37:16 +08:00
    这个同事可能这一周都在摸鱼
    vertigo
        50
    vertigo  
       2019-05-29 17:47:41 +08:00 via iPhone
    遇到将 git 当“云同步”来进行本地-》服务器传代码的同事才头大
    linvaux
        51
    linvaux  
       2019-05-29 18:43:02 +08:00 via Android
    规定下班前必须提交代码。不然就扣绩效。次数多了你看他老不老实
    huluhulu
        52
    huluhulu  
       2019-05-29 19:28:44 +08:00 via iPhone
    楼主显然不是大公司的…哪有大公司这么频繁提交的啊…提交一般提交到分支,然后回归 case 什么自动化都跑过了。针对这块代码的需求 test case 都跑过了,才能合到 master …然后每天触发 daily …测试部门拿 daily 测出问题,就是 bug,都是要报问题单的,问题单流程很 heavy。谁没事老往 master 提代码啊…
    SEARCHINGFREE
        53
    SEARCHINGFREE  
       2019-05-29 19:31:54 +08:00
    切分支不是随便玩
    zgl263885
        54
    zgl263885  
       2019-05-30 07:04:17 +08:00 via iPhone
    扣绩效呗!
    liangweijia6000
        55
    liangweijia6000  
       2019-05-30 10:01:37 +08:00
    ??
    我咋没看懂,一周提交一次代码有什么问题?
    我一个功能写一半,提交了编译都通不过也要提交的么?
    pink123
        56
    pink123  
    OP
       2019-05-30 10:01:46 +08:00
    @yalin 哈哈一个功能写了一周,也不是很复杂的那种
    yalin
        57
    yalin  
       2019-05-30 14:07:31 +08:00
    @pink123 给个确定的时间,说明你需要什么时候之前提交测试呗
    Linxing
        58
    Linxing  
       2019-05-31 09:36:39 +08:00 via iPhone
    难得不是吗 一周一个 feature 肯定只有一个提交啊 最多拆分一下 做一点 review 一点
    amon
        59
    amon  
       2019-05-31 11:26:05 +08:00
    可能之前是用 svn 的,让他用 git,保持时不时 commit 的习惯,即使不 push 到服务器,提交到本地仓库也算版本管理啊。
    pink123
        60
    pink123  
    OP
       2019-07-22 16:25:52 +08:00
    @yalin 结果就悲剧了,哈哈也不知道什么问题了,是能力问题还是什么不知道了
    pink123
        61
    pink123  
    OP
       2019-07-22 16:26:35 +08:00
    @amon 可能是以前经常一个人干吧,自己天王老子想干什么就干什么
    pink123
        62
    pink123  
    OP
       2019-07-22 16:28:55 +08:00
    @liangweijia6000 不提交就看不出工作量啊,所以最后悲剧了,最后跑路啦
    pink123
        63
    pink123  
    OP
       2019-07-22 16:29:38 +08:00
    @lygmqkl 作为程序员版本控制是应该的,每天下班前提交代码没有错啊
    pink123
        64
    pink123  
    OP
       2019-07-22 16:30:17 +08:00
    @zyp0921 怎么可能呢,这样的话不是太水了吧一天挑个编译错误搞几天呀
    pink123
        65
    pink123  
    OP
       2019-07-22 16:30:51 +08:00
    @lygmqkl 至少下班前把今天的事情提交上去,这是很 basic
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5294 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 58ms · UTC 09:26 · PVG 17:26 · LAX 01:26 · JFK 04:26
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.