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

大家公司对提交代码都有什么限制要求,好奇了解下?

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

    我们现在要求每一行代码的 commit 信息都要关联上任务编号,然后代码量+任务复杂度等来计算开发效率。

    59 条回复    2021-11-01 10:31:01 +08:00
    Jwyt
        1
    Jwyt  
       87 天前
    一样 任务号 + perf/fix/feat + ...
    wu67
        2
    wu67  
       87 天前
    要按提交内容的类型区分, 还要抓个人过来人肉 review, 然后把对方名字写进 commit msg...
    cppc
        3
    cppc  
       87 天前
    意思是不分任务就不能提代码,开发人员干活只认任务编号不认人?
    shanghai1943
        4
    shanghai1943  
       87 天前
    好奇,有多少公司会计算开发效率的?之前貌似没怎么碰到过。
    wenqiang1208
        5
    wenqiang1208  
       87 天前   ❤️ 1
    开发效率 本来就没办法量化的, 对代码的熟悉程度,复杂度,个人等级都有关系的
    ElegantOfKing
        6
    ElegantOfKing  
       87 天前
    comit 限定信息:任务号 fix or feature 提交详情
    MuscleOf2016
        7
    MuscleOf2016  
    OP
       87 天前
    @cppc 现在造成这种恶性结果了,有的小组长已经指挥不动手下人了,小改动,就要任务编号。然后只能组长自己改。选择忽略扫描的 commit msg
    NillSpake
        8
    NillSpake  
       87 天前
    觉得没什么大问题吧,我们用的 jira ,所有任务分配任务编号 TM-XX ,优先级。
    所有任务开分支命名 姓名 /TM-XX-任务描述 ,commit 也是如此,代码注释也是如此。
    NillSpake
        9
    NillSpake  
       87 天前
    而且本身,我们修改一个 button rename 都会有一个任务。
    但是以此作为来计算开发效率,就开始卷了。
    cyrivlclth
        10
    cyrivlclth  
       87 天前
    没啥要求,就一个 commit 模板,feat\fix\chore\refactor+Closes#Issue
    joesonw
        11
    joesonw  
       87 天前
    x940727
        12
    x940727  
       87 天前
    @MuscleOf2016 这样挺好的啊,顶多就是小组长麻烦点,多拿钱多干事呗,大不了给老板说不写代码了就专门管这些也不是不行啊。
    FallenMax
        13
    FallenMax  
       87 天前
    hccsoul
        14
    hccsoul  
       87 天前
    没有 ..随便提交
    coderluan
        15
    coderluan  
       87 天前   ❤️ 1
    没限制,有些同事连 git 是干啥的都不知道,完全是当网盘用的。

    有次让一大哥先上传个原始版本,再把最终版本提交上去,结果大哥建了两个文件夹,一个原始版本,一个最终版本。至于各种大号 bin/dll 都往上传也是常态,我说你这样再 clone 会很麻烦,他们说那我删了呗,草。
    cuzfinal
        16
    cuzfinal  
       87 天前
    没有要求,随便搞。
    legiorange
        17
    legiorange  
       87 天前
    Jira 号 + Jira 号的 title + 描述

    代码量+任务复杂度等来计算开发效率不现实
    zqx
        18
    zqx  
       87 天前 via Android
    github 很多项目都是注释里带着 issue 链接
    6364v2
        19
    6364v2  
       87 天前
    commit 信息:更新
    dangyuluo
        20
    dangyuluo  
       87 天前   ❤️ 7
    不许有脏话
    cndydb
        21
    cndydb  
       87 天前
    可以为空
    chaodada
        22
    chaodada  
       87 天前   ❤️ 5
    commit 信息:123
    commit 信息:第一次提交
    commit 信息:测试
    commit 信息:1

    commit 信息:修复 xxx
    commit 信息:修复 xxxxxx
    Yuan2One
        23
    Yuan2One  
       87 天前
    [issus/需求 US/问题单号][feat/fix/docs/style/refactor/test/chore] 功能变更描述
    andyskaura
        24
    andyskaura  
       87 天前
    jdhao
        25
    jdhao  
       87 天前 via Android
    @chaodada 过于真实😅
    oppoic
        26
    oppoic  
       87 天前
    不准打 “1111” 或者 “---” 之类的
    wellsc
        27
    wellsc  
       87 天前
    commit 也这么严吗? merge request 或者 branch 严格一点还能接受
    openmm
        28
    openmm  
       87 天前
    啥是任务号?
    chenzheyu
        29
    chenzheyu  
       87 天前
    代码量?写业务不行,写废话不是很简单的事情吗?
    chenyi
        30
    chenyi  
       87 天前
    commit 的用户名必须要跟自己的用户名一致(
    xnth97
        31
    xnth97  
       87 天前
    commit 时 CI 自动跑一些东西,比如 linter 检查 code style 、precommit 跑一遍测试、postcommit 跑一遍测试
    SmaliYu
        32
    SmaliYu  
       87 天前
    不许写拼音……
    otakustay
        33
    otakustay  
       87 天前
    @MuscleOf2016 组长没有权限创建一个任务吗?
    JoeBreeze
        34
    JoeBreeze  
       87 天前
    我们组随便, 必要的时候能找到人背锅就好
    ckaock
        35
    ckaock  
       87 天前
    换老板
    g190875565
        36
    g190875565  
       87 天前
    @chaodada #22 真实,看笑了
    isBitter
        37
    isBitter  
       87 天前
    https://github.com/streamich/git-cz 加了个 pre-commit-hook
    另外大概的 review 下命名,代码组织...
    至于 merge 或 rebase 没要求。
    pengtdyd
        38
    pengtdyd  
       87 天前
    直接分模块开发不就好了,哪这么麻烦,小公司还弄一套繁琐的流程,最后自己累死自己
    iovekkk
        39
    iovekkk  
       87 天前
    之前的公司,打通了需求录入系统、开发 gitlab 以及缺陷管理系统
    从产品录入一个需求开始,转到研发这里,确认之后会自动创建一个分支
    研发在这个分支上开发完成,在系统上确认完成以后,会自动打包然后提测,流程就到测试那里去了
    测试验收以后,这个功能分支又会自动合并到开发分支上
    0Vincent0Zhang0
        40
    0Vincent0Zhang0  
       87 天前 via Android
    任务复杂度怎么考虑的?
    大家按最简单的方法完成?
    不考虑维护和扩展性?
    需求不提边界就尽量不考虑校验?
    这种 KPI 只会扭曲结果。
    rgxiao
        41
    rgxiao  
       87 天前
    要求是只要能跑起来, 不管是三条腿跑起来的还是一条腿跑起来的, 反正能跑起来就可以了.
    haozheliu
        42
    haozheliu  
       87 天前
    必须是公司邮箱,加上 jira 的 issue 号
    mmrindextt
        43
    mmrindextt  
       87 天前
    还是要有一定的规范,写起来才会舒服。
    Merlini
        44
    Merlini  
       87 天前
    JIRA 号和内容概括
    k9982874
        45
    k9982874  
       87 天前 via Android
    @coderluan 巡检 repo 时发现 ios 组的 repo 高达 1 个 g ,一看 ios 的同学把 pods 提交上来了,就离谱
    Linxing
        46
    Linxing  
       87 天前
    @coderluan 哈哈哈 笑出了声音 他怎么找到工作的
    zzlatan
        47
    zzlatan  
       87 天前
    真羡慕你们业务这么闲。。。
    yyysuo
        48
    yyysuo  
       87 天前
    @dangyuluo 没人性,干活还不让发泄了。
    skiy
        49
    skiy  
       87 天前
    @Yuan2One 这个好像是 Angular 的规范。
    rannnn
        50
    rannnn  
       87 天前
    - 每一个 commit 都要单独 review
    - 只 rebase 不 branch 不 merge
    - 因为没有 branch 提交到 master 要排队 rebase
    Rache1
        51
    Rache1  
       87 天前
    PR 时要有任务编号,标题尽量一句话描述清楚。

    我用 alias 做了 commit 自动获取分支名上的任务编号填到 commit 中去,所以 commit 里面也就有了,主要是方便后期定位是哪个需求造成的改动。
    anxxv2
        52
    anxxv2  
       87 天前
    @chaodada 看到这 没忍住 哈哈
    clf
        53
    clf  
       87 天前 via Android
    gitmoji + issue + 说明
    sagaxu
        54
    sagaxu  
       86 天前 via Android
    曾经试行过,不但要有 task id ,每个 task 在进度表上不能超过半天,超过半天的都要拆分细化,周报中至少要有 10 个 task 。如果有 1 小时以上的会议,会议也要提一个 task ,否则时间对不上。
    Nich0la5
        55
    Nich0la5  
       86 天前
    原则上 commit 肯定要带编号的 系统要管理到这次改动影响哪个 bug 或者关联哪次迭代,(虽然我们这边有挺多人 commit 挺随意的就是了)
    darkengine
        56
    darkengine  
       86 天前
    commit 信息都要关联上任务编号 (JIRA issue/task ID)属于基本操作了吧,以后查问题(找人背锅)也好找。
    Yuan2One
        57
    Yuan2One  
       86 天前
    @skiy 不清楚,我们这边是 CIE 定义的,相对应的任务会关联到需求,问题单等相应的流程上
    susecjh
        58
    susecjh  
       85 天前
    Feature/FIx/... + 描述即可
    hardwork
        59
    hardwork  
       84 天前
    @coderluan git 不太适合二进制大文件存储,但是有些项目确实需要存储二进制,你们怎么弄的
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1351 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 32ms · UTC 18:26 · PVG 02:26 · LAX 10:26 · JFK 13:26
    ♥ Do have faith in what you're doing.