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

同事直接往 master 推未经测试的代码

  •  
  •   justR · 50 天前 via Android · 7862 次点击
    这是一个创建于 50 天前的主题,其中的信息可能已经有所发展或是发生改变。
    最近新来个同事,代码写了就直接推到 master, 一堆 bug,说了也不听,遇到这种人怎么办
    61 条回复    2024-02-27 16:35:45 +08:00
    ChefIsAwesome
        1
    ChefIsAwesome  
       50 天前   ❤️ 16
    恕我直言,这都没人管的公司也就别在那待了吧。
    dfkjgklfdjg
        2
    dfkjgklfdjg  
       50 天前
    按规定办事咯。没有规定的话🤷‍♂️,得看你是他同事还是领导了。
    duange7X24
        3
    duange7X24  
       50 天前
    谁出问题谁担责。如果你是他领导,屡教不改直接滚蛋;如果是同事,说了也不听,那就把火烧大
    TArysiyehua
        4
    TArysiyehua  
       50 天前   ❤️ 8
    这个是你们开发规范的问题吧,跟你同事关系不大。
    首先你们有项目文档吗?有没有说开发流程是怎么样的?比如必须先经过测试,通过了才能提交。代码覆盖于要多少,代码风格要怎么样?
    其次,你们为什么没有对 master 进行限制与保护,新人可以直接往 master 推送本来就是很离谱的事情。
    最后,你们的 review 机制是怎么样的?


    这些东西你都不说,就说人家说了也不听。固然,一个对我自然要求较高的程序员,每次提交 的代码都是经过充分测试的,但是这个明显是考验每个程序员的自我修养。在很多公司个团队,靠自觉,本身就是个笑话
    EdmondGUO
        5
    EdmondGUO  
       49 天前
    听起来不像什么正经公司
    alexHong
        6
    alexHong  
       49 天前   ❤️ 8
    他为什么能推上去。
    akira
        7
    akira  
       49 天前
    git 流程 做一下规范。。
    chendl111
        8
    chendl111  
       49 天前
    你们没有代码测试和代码复核的人员吗?流程上的问题不要怪人
    kirito41dd
        9
    kirito41dd  
       49 天前 via iPhone   ❤️ 3
    你们主分支能直接 push 也挺抽象的(
    Helsing
        10
    Helsing  
       49 天前 via iPhone
    找领导把 master 保护起来
    hello2090
        11
    hello2090  
       49 天前   ❤️ 1
    不是,等一下,一堆 bug 怎么了,谁的 code 没有 bug? 你 push 到 master 的没有 bug? 既然你们能直接 push 到 master 那他做的就没错啊。
    expkzb
        12
    expkzb  
       49 天前
    他推了,但还是应该有人负责审核和 merge 的吧
    anjingdexiaocai
        13
    anjingdexiaocai  
       49 天前 via Android
    这种就别怪你同事了,代码流程就有问题。
    kisick
        14
    kisick  
       49 天前 via iPhone
    公司的问题
    ivvei
        15
    ivvei  
       49 天前
    可以锁分支啊,为什么不锁。你们本来就没啥规范吧是不是?
    shimmerh
        16
    shimmerh  
       49 天前 via iPhone
    这能推 master 本身说明公司不管控代码啊。。。这不是组长干的吗?
    joyhub2140
        17
    joyhub2140  
       49 天前
    master 属于保护分支,只有管理员才可以推送,其余组员一律只能推 merge request 。
    ytmsdy
        18
    ytmsdy  
       49 天前
    锁分支不就完了么。不允许直接 push 代码到 main 分支。多大点事情
    gosky
        19
    gosky  
       49 天前
    小心根本不是正经公司,不可能树立规范
    如果不幸是的,赶紧跑路
    iyaozhen
        20
    iyaozhen  
       49 天前
    我们 push 不了,不然新人来了不知道,很容易误操作呀
    a1b2c3T
        21
    a1b2c3T  
       49 天前 via iPhone
    @alexHong 好多小公司是没有 review 这一步的,也没有分支管理,见多不怪了🤦‍♂️
    vitoliu
        22
    vitoliu  
       49 天前
    我就经常往 master 推代码,因为我们是特性分支出包,所以 master 就是 dev~
    greatghoul
        23
    greatghoul  
       49 天前
    啥公司管理这么混乱,pr review 都没有 😂
    NickX
        24
    NickX  
       49 天前
    为什么 master 能随便 push ,收回权限使用 Merge Request 吧
    1647846714
        25
    1647846714  
       49 天前
    master 可以设置 push 权限的。
    iseki
        26
    iseki  
       49 天前
    小公司吧,甚至一个仓库就一个人自己玩?那流程什么的有时候就多余了,自然是想怎么搞怎么搞,push -f 都会是家常便饭。
    dxk611
        27
    dxk611  
       49 天前
    这种人肯定有问题,但是最大问题在公司管理上
    ST0RMTR00PER
        28
    ST0RMTR00PER  
       49 天前
    为什么要给他权限?
    sarices
        29
    sarices  
       49 天前
    不知道你的同事什么水平,但你们项目管理负责人应该负最大的责任
    phrack
        30
    phrack  
       49 天前 via iPhone   ❤️ 14
    楼主暴露了同事的水平的同时,也暴露了自己的水平,领导的水平,还有公司的水平。
    warcraft1236
        31
    warcraft1236  
       49 天前   ❤️ 1
    你能相信我们几千人的全球业务的公司,master 分支其实和 dev 分支差不多,每天都有人往上边推不是足够稳定的代码,所以我经常遇到的问题就是,拉了最新的 master 分支的代码->编译报错->GG
    warcraft1236
        32
    warcraft1236  
       49 天前
    @warcraft1236 几千人指的是我所在的大项目的组,不是整个公司
    SekiBetu
        33
    SekiBetu  
       49 天前
    出了问题有人负责就行了
    kenvix
        34
    kenvix  
       49 天前
    快跑
    Adelell
        35
    Adelell  
       49 天前 via iPhone
    真正的大神都是直接 push main ,只是人家没 bug 。
    mastxx ,慎用,影响仕途。
    leaflxh
        36
    leaflxh  
       49 天前
    小心离职后成为下一个 bilibili-master
    jayjoe
        37
    jayjoe  
       49 天前
    没有 git 流程规范管理是相当可怕的一件事。。
    Mirage09
        38
    Mirage09  
       49 天前
    branch protection 很重要
    我自己也好几次手滑往 master branch push ,但是因为有保护都被 reject ,这才合理
    wm5d8b
        39
    wm5d8b  
       49 天前 via Android
    “培训班老师说了,软件开发就是写代码”🤪
    新同事不都是这样讲的吗,测试是什么?培训班没教过
    allgy
        40
    allgy  
       49 天前
    你们 master 是任何人都能推送的吗?
    manhere
        41
    manhere  
       49 天前
    拿 git 当 ftp 用的公司不在少数
    xinyu391
        42
    xinyu391  
       49 天前
    把 master 推送权限关了,
    他只能推送分支,然后提交 merge 请求,
    lilei2023
        43
    lilei2023  
       49 天前
    小公司,没啥稀奇的,都是一个人搞,我这还有人直接在把 dev 当作 master 再用! master 都落后 n 多版本了!
    felmoon
        44
    felmoon  
       49 天前
    master 咋能推上去的呢?
    jydeng
        45
    jydeng  
       49 天前
    权限太大了,能直接推 master
    huwenzhe
        46
    huwenzhe  
       49 天前
    他既然都能推了,还有什么问题么
    tairan2006
        47
    tairan2006  
       49 天前
    master protect 一下不就行了
    changepll
        48
    changepll  
       49 天前
    这有啥的. 我同事把 master,develop,还有一个功能分支来回 merge. 说多了还给你甩脸色. 我懒得说了.
    有事就赖他身上. 挺好的
    xx6412223
        49
    xx6412223  
       49 天前
    说实话,这和之前传言青岛啤酒有人往原材料里尿尿一样
    一个懒人+一个草台班子
    deltaone
        50
    deltaone  
       49 天前
    用制度 (git setting) 来防止犯错。
    eshezhao
        51
    eshezhao  
       49 天前
    @akira 这是听到过最好的建议
    snoy
        52
    snoy  
       48 天前
    我的规矩就是规矩,整些没用都是瞎扯,你是 leader 的情况下可以,其他的无所谓直接向上反映。
    351994
        53
    351994  
       48 天前
    能直接推 master 的也确实绷不住了
    MuscleOf2016
        54
    MuscleOf2016  
       48 天前
    如果这个库,和你没啥关系,随便他弄就行了。如果这个库和你有关系,就别让他瞎搞。
    bruce0
        55
    bruce0  
       48 天前
    像楼上说的, 锁一下 master 分支就行了

    我们用 svn, 以前总有人不 fmt 代码就提交, 后来我写了一个脚本, 检查代码格式, 格式有问题 直接 拒绝提交,从此没有这个烦恼了
    ExplodingFKL
        56
    ExplodingFKL  
       48 天前
    上面不管那你也不管,只要把强制提交关闭就行,到时候别背锅的都找不到
    colorcat
        57
    colorcat  
       48 天前
    如果你们团队使用主线开发模型,那就是应该往 master 分支推送啊
    zhanshen1614
        58
    zhanshen1614  
       48 天前
    任何人可以合并到主分支说明流程不规范,应锁定生产环境的分支,合并要走合并请求不能线下合并推到线上。弄个 git 流程不然太乱了。
    zhanshen1614
        59
    zhanshen1614  
       48 天前
    @jayjoe 非常可怕。我之前在一个项目组没有分支规范堆积数百上千个分支,有人看走眼删了一个没开发完的分支碰巧遇上重启任务导致整个重写。我给上级建议弄个 git 分支规范还被反驳:“你要关注业务产出!领导的决策要兼顾大多数团队,其他组的人都没意见为什么就你有意见?”。
    l4ever
        60
    l4ever  
       47 天前
    你自己不设置分支保护怪人家咯?
    新同事可能他以前自己写一个项目, 就直接推 master 了呗,
    你要么保护起来, 要么告诉他不要推呗
    Aruok
        61
    Aruok  
       47 天前
    还是项目流程不规范啊.
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5965 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 41ms · UTC 03:24 · PVG 11:24 · LAX 20:24 · JFK 23:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.