公司统计代码行数的逆天逻辑

2023-12-28 15:23:29 +08:00
 sqfphoenix

之前发了个帖子吐槽公司统计代码行数 前情提要

经过我两个月的奋斗,各种水行数,甚至突破自己底线贴工具类到代码里面,发现怎么都比不过别人的代码行数,今天才发现,原来公司统计是会把 merge 提交的行数也全部计算进去,而我每次拉代码都用的 rebase ,怪不得我比不过别人啊,合着我写的代码全都算到别人的数据里面去了

从今天开始,继续突破自己底线,每次提交必选 merge ,反正 git 提交历史已经是一个粪坑了

11953 次点击
所在节点    职场话题
84 条回复
wolfan
2023-12-29 13:42:54 +08:00
贵司领导可能觉得这样装逼吧,毕竟领导就爱 PUA 式管理。总得显得自己有能力驾驭尔等高级农民工的。
Felldeadbird
2023-12-29 13:47:34 +08:00
能够把代码行数来统计工作效率,说明公司想裁人,恶心人罢了。

根据之前类似贴子的回复,把那些包的代码 cv 复制一份。其次,循环全改手写。
Yuanlaoer
2023-12-29 13:49:24 +08:00
@nevin47 您想表达的重点是?
Yuanlaoer
2023-12-29 13:52:12 +08:00
@iyaozhen 那您觉得这个指标怎么样?
ryan4290
2023-12-29 13:57:18 +08:00
@chihiro2014 太二逼了哈哈哈
solitude511
2023-12-29 14:15:01 +08:00
哈哈哈哈
好奇今天改一个目录明天再改回来是不是也算
Varobjs
2023-12-29 14:20:24 +08:00
我一直习惯 git pull --rebase origin

原来 git merge 还有这 “优点”,涨姿势了
BeautifulSoap
2023-12-29 14:21:26 +08:00
@nothingistrue 我十分怀疑你有没有用过 Github
Varobjs
2023-12-29 14:22:00 +08:00
@shalk 然后有 bug 永远看最后一个提交是谁,不管改动的是啥,阁下如何应对
nothingistrue
2023-12-29 14:25:23 +08:00
@horizon #56 哦,原来是这个首先源分支上 「 rebase --no-ff 」,然后目标分支上 「 merge --ff-only 」,如果不能 rebase --no-ff 还不让做的 Github 专有的脑残方式。这个脑残方式是一定要禁止的,但请注意它并不是通常所说的 rebase merge 。

通常所说的 rebase merge ,是首先源分支上 「 rebase 」,然后目标分支上「 merge --no-ff 」,如果有冲突则可通过界面处理,这样的过程。Gitlab 、微软 Devops 、Gitea 等都支持这种方式,就 Github 出于提交签名的考虑,不支持这种方式。
4771314
2023-12-29 14:33:54 +08:00
@jasonyang9 手下的人未必都是精英,但是领导是真的智障
BQsummer
2023-12-29 14:51:35 +08:00
我们公司用 commit id 去重后算 commit 里的代码行数,你这种 rebase 的会算重,代码量大增,来我们公司吧 [狗头]
iSecret
2023-12-29 15:04:16 +08:00
你们是用的 Gitlab 吗?我司前段时间统计项目是从 Gitlab 拉取的 Merge 日志来统计影响行,然后有项目组是直接 Push 到 master 分支的,导致月均代码行数小于 1 行,笑死了。
pkokp8
2023-12-29 15:09:59 +08:00
奇数次提交 unix2dos
偶数次提交 dos2unix😁
volCan0
2023-12-29 15:22:29 +08:00
@IamLostInCoding 干着不累么
KgM4gLtF0shViDH3
2023-12-29 15:32:59 +08:00
@Dogtler 一个月不发工资就应该慌了,你怎么忍得住的
iyaozhen
2023-12-29 19:58:56 +08:00
@Yuanlaoer 我是 QA ,我不反对这个指标。怎么说呢,质量可视,总得需要一些指标
chihiro2014
2023-12-29 22:41:05 +08:00
@ryan4290 技术总监是日本回来的废物,跟国内起码脱节 10 年,笑死
Yuanlaoer
2023-12-30 01:22:14 +08:00
@iyaozhen 您这 2 条回复总结在一起,我帮你翻译成简单的普通话吧:“不认同但不想明说”
nevin47
2024-01-03 10:44:11 +08:00
@Yuanlaoer #63 千行缺陷密度在项目实操中很好用,用来评估项目质量是一个很合理的指标。类似你说的清空 ignore 这种操作,在 PR 的时候有 Maintainer 来拦截的,不会影响缺陷密度统计

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1004126

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX