• 请不要在回答技术问题时复制粘贴 AI 生成的内容
MuscleOf2016
0D
V2EX  ›  程序员

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

  •  
  •   MuscleOf2016 · Oct 29, 2021 · 8978 views
    This topic created in 1831 days ago, the information mentioned may be changed or developed.

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

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

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

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

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

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