2

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

 2 years ago
source link: https://www.v2ex.com/t/811425
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

V2EX  ›  程序员

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

  MuscleOf2016 · 1 天前 · 5286 次点击

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

57 条回复    2021-10-30 14:27:55 +08:00

Jwyt

Jwyt   1 天前

一样 任务号 + perf/fix/feat + ...

wu67

wu67   1 天前

要按提交内容的类型区分, 还要抓个人过来人肉 review, 然后把对方名字写进 commit msg...

cppc

cppc   1 天前

意思是不分任务就不能提代码,开发人员干活只认任务编号不认人?

shanghai1943

shanghai1943   1 天前

好奇,有多少公司会计算开发效率的?之前貌似没怎么碰到过。

wenqiang1208

wenqiang1208   1 天前   ❤️ 1

开发效率 本来就没办法量化的, 对代码的熟悉程度,复杂度,个人等级都有关系的

ElegantOfKing

ElegantOfKing   1 天前

comit 限定信息:任务号 fix or feature 提交详情

MuscleOf2016

MuscleOf2016   1 天前

@cppc 现在造成这种恶性结果了,有的小组长已经指挥不动手下人了,小改动,就要任务编号。然后只能组长自己改。选择忽略扫描的 commit msg

NillSpake

NillSpake   1 天前

觉得没什么大问题吧,我们用的 jira ,所有任务分配任务编号 TM-XX ,优先级。
所有任务开分支命名 姓名 /TM-XX-任务描述 ,commit 也是如此,代码注释也是如此。

NillSpake

NillSpake   1 天前

而且本身,我们修改一个 button rename 都会有一个任务。
但是以此作为来计算开发效率,就开始卷了。

cyrivlclth

cyrivlclth   1 天前

没啥要求,就一个 commit 模板,feat\fix\chore\refactor+Closes#Issue

joesonw

joesonw   1 天前

x940727

x940727   1 天前

@MuscleOf2016 这样挺好的啊,顶多就是小组长麻烦点,多拿钱多干事呗,大不了给老板说不写代码了就专门管这些也不是不行啊。

FallenMax

FallenMax   1 天前

hccsoul

hccsoul   1 天前

没有 ..随便提交

coderluan

coderluan   1 天前   ❤️ 1

没限制,有些同事连 git 是干啥的都不知道,完全是当网盘用的。

有次让一大哥先上传个原始版本,再把最终版本提交上去,结果大哥建了两个文件夹,一个原始版本,一个最终版本。至于各种大号 bin/dll 都往上传也是常态,我说你这样再 clone 会很麻烦,他们说那我删了呗,草。

cuzfinal

cuzfinal   1 天前

没有要求,随便搞。

legiorange

legiorange   1 天前

Jira 号 + Jira 号的 title + 描述

代码量+任务复杂度等来计算开发效率不现实

zqx

zqx   1 天前 via Android

github 很多项目都是注释里带着 issue 链接

6364v2

6364v2   1 天前

commit 信息:更新

dangyuluo

dangyuluo   1 天前   ❤️ 7

不许有脏话

cndydb

cndydb   1 天前

chaodada

chaodada   1 天前   ❤️ 3

commit 信息:123
commit 信息:第一次提交
commit 信息:测试
commit 信息:1

commit 信息:修复 xxx
commit 信息:修复 xxxxxx

Yuan2One

Yuan2One   1 天前

[issus/需求 US/问题单号][feat/fix/docs/style/refactor/test/chore] 功能变更描述

jdhao

jdhao   1 天前 via Android

@chaodada 过于真实😅

oppoic

oppoic   1 天前

不准打 “1111” 或者 “---” 之类的

wellsc

wellsc   1 天前

commit 也这么严吗? merge request 或者 branch 严格一点还能接受

openmm

openmm   1 天前

啥是任务号?

chenzheyu

chenzheyu   1 天前

代码量?写业务不行,写废话不是很简单的事情吗?

chenyi

chenyi   1 天前

commit 的用户名必须要跟自己的用户名一致(

xnth97

xnth97   1 天前

commit 时 CI 自动跑一些东西,比如 linter 检查 code style 、precommit 跑一遍测试、postcommit 跑一遍测试

SmaliYu

SmaliYu   1 天前

不许写拼音……

JoeBreeze

JoeBreeze   1 天前

我们组随便, 必要的时候能找到人背锅就好

ckaock

ckaock   1 天前

g190875565

g190875565   23 小时 57 分钟前

@chaodada #22 真实,看笑了

isBitter

isBitter   23 小时 8 分钟前

https://github.com/streamich/git-cz 加了个 pre-commit-hook
另外大概的 review 下命名,代码组织...
至于 merge 或 rebase 没要求。

pengtdyd

pengtdyd   23 小时 1 分钟前

直接分模块开发不就好了,哪这么麻烦,小公司还弄一套繁琐的流程,最后自己累死自己

iovekkk

iovekkk   22 小时 50 分钟前

之前的公司,打通了需求录入系统、开发 gitlab 以及缺陷管理系统
从产品录入一个需求开始,转到研发这里,确认之后会自动创建一个分支
研发在这个分支上开发完成,在系统上确认完成以后,会自动打包然后提测,流程就到测试那里去了
测试验收以后,这个功能分支又会自动合并到开发分支上

0Vincent0Zhang0

0Vincent0Zhang0   22 小时 45 分钟前 via Android

任务复杂度怎么考虑的?
大家按最简单的方法完成?
不考虑维护和扩展性?
需求不提边界就尽量不考虑校验?
这种 KPI 只会扭曲结果。

rgxiao

rgxiao   22 小时 41 分钟前

要求是只要能跑起来, 不管是三条腿跑起来的还是一条腿跑起来的, 反正能跑起来就可以了.

haozheliu

haozheliu   22 小时 39 分钟前

必须是公司邮箱,加上 jira 的 issue 号

mmrindextt

mmrindextt   21 小时 50 分钟前

还是要有一定的规范,写起来才会舒服。

Merlini

Merlini   21 小时 44 分钟前

JIRA 号和内容概括

k9982874

k9982874   20 小时 35 分钟前 via Android

@coderluan 巡检 repo 时发现 ios 组的 repo 高达 1 个 g ,一看 ios 的同学把 pods 提交上来了,就离谱

Linxing

Linxing   20 小时 34 分钟前

@coderluan 哈哈哈 笑出了声音 他怎么找到工作的

zzlatan

zzlatan   20 小时 13 分钟前

真羡慕你们业务这么闲。。。

yyysuo

yyysuo   20 小时 11 分钟前

@dangyuluo 没人性,干活还不让发泄了。

skiy

skiy   17 小时 24 分钟前

@Yuan2One 这个好像是 Angular 的规范。

rannnn

rannnn   16 小时 17 分钟前

- 每一个 commit 都要单独 review
- 只 rebase 不 branch 不 merge
- 因为没有 branch 提交到 master 要排队 rebase

Rache1

Rache1   15 小时 30 分钟前

PR 时要有任务编号,标题尽量一句话描述清楚。

我用 alias 做了 commit 自动获取分支名上的任务编号填到 commit 中去,所以 commit 里面也就有了,主要是方便后期定位是哪个需求造成的改动。

anxxv2

anxxv2   13 小时 39 分钟前

@chaodada 看到这 没忍住 哈哈

clf

clf   13 小时 31 分钟前 via Android

gitmoji + issue + 说明

sagaxu

sagaxu   8 小时 16 分钟前 via Android

曾经试行过,不但要有 task id ,每个 task 在进度表上不能超过半天,超过半天的都要拆分细化,周报中至少要有 10 个 task 。如果有 1 小时以上的会议,会议也要提一个 task ,否则时间对不上。

Nich0la5

Nich0la5   4 小时 4 分钟前

原则上 commit 肯定要带编号的 系统要管理到这次改动影响哪个 bug 或者关联哪次迭代,(虽然我们这边有挺多人 commit 挺随意的就是了)

darkengine

darkengine   4 小时 0 分钟前

commit 信息都要关联上任务编号 (JIRA issue/task ID)属于基本操作了吧,以后查问题(找人背锅)也好找。

Yuan2One

Yuan2One   44 分钟前

@skiy 不清楚,我们这边是 CIE 定义的,相对应的任务会关联到需求,问题单等相应的流程上

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK