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

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

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

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

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

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

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

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

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

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

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