挺生气的,关于领导 git 管理的一顿臭骂

2023-07-04 22:43:44 +08:00
 breadykidliu

事情是这样的
目前项目发生产的分支没有做 protect ,任何人都能往上 push
(之前提议过生产分支要做 protect ,某领导以每次合并都要由某人审核太麻烦被拒)
组里定的规矩(不成文,口口相传,也不知道我掌握的是否全面)是:迭代发生产时,将功能代码合上生产分支
今天突然要发个 hotfix ,看 commit 记录发现了我提交的某个 bugfix
tl 直接质问我为什么当天不发版的内容要合上生产分支,
我说这个 bug 拖一天生产上每日生成的文件名就错一天,肯定越早上线修复越好
然后他开始 balabala 一顿说
我觉得你们自己不做限制,规定又不成文,就别怪其他人往上合,
单个无依赖的 commit 你实在不允许上线,你 drop 掉就行了,完事后和当事人说下,强调下规定,就可以了,
你冲我 balabala 叫个 p 啊,大煞笔!

11292 次点击
所在节点    程序员
91 条回复
yule111222
2023-07-05 09:10:11 +08:00
LS 几个人别装逼了,生产分支不 protect 能直接 push 显然是技术管理问题,别怪小弟
dqzcwxb
2023-07-05 09:14:39 +08:00
做错了事还嘴硬
hokori
2023-07-05 09:16:46 +08:00
我告诉你,我之前公司的一个总监,git 不会,linux 不懂,照样月入 5w+。 薪资是有个同事泡了会计,不过那个总监 5 个月就走了,应该是劝退的。
sujin190
2023-07-05 09:17:44 +08:00
严格来说这样确实不对,口头约定也是规则,现在出来很多规则外也还有软规则,之所以这样选择完全是基于沟通协调协作成本考量,类比于现实法律这个规则并不会也不能事事俱到,我们也还要遵守很多道德伦理方面的规则,再说吧如果你处在一个事事都有硬规则,否则大家就钻空子的工作环境,你也会觉得很非常难受

比如这件事情中,发布分支不强制限制不可推送确实存在可以推送的可能,但是你在项目组提前说一说或者和 leader 提前说清楚再提交,和你直接不管不顾提交两者是完全不同的,想着更快修复问题是好事,好心要有好路径去完成事情才会有好结果,一般逻辑中人家把这个称为情商
QlanQ
2023-07-05 09:18:10 +08:00
一般不都是 protected 然后走 merge 么
ql562482472
2023-07-05 09:20:09 +08:00
我建议你立刻去和领导道歉,就说
“领导,我意识到这个做法是错误的了,即使没有 protect ,我也不应将没规划的内容提交到版本分支上。”

这样你的意图能够达到(加 protect ),在领导那边的印象分也能赚回来。

当然的建议是我比较功利了,但是我觉得对你真的有好处
8355
2023-07-05 09:21:20 +08:00
你们没看懂,op 是在倒逼负责人开启生产分支 protect 用自己的方式保护生产分支👍
kalman03
2023-07-05 09:22:29 +08:00
在高效的小团队里面,约束的效率没有约定高!
lambdaq
2023-07-05 09:26:15 +08:00
LZ 还年轻吧。。。。这领导是对你好。。。。。。。。。

等你变职场老油条了,你也会不可救药的避免担责走死板流程的。。。
wolfie
2023-07-05 09:27:51 +08:00
@yule111222
看不懂上下文少说两句,任何 git 问题都可以推脱给 release 没 protect 是吧。
akring
2023-07-05 09:28:20 +08:00
「我觉得你们自己不做限制,规定又不成文,就别怪其他人往上合」

你看,其实你也知道这样不好,只不过是想把锅甩到没规定上罢了
GOOD21
2023-07-05 09:28:54 +08:00
一般规范都是出了事故才订立起来的。只有经历过一次,才知道什么叫“敬畏上线”。
polo3584
2023-07-05 09:30:48 +08:00
小团队确实直接 push 效率高,不过 push 截图到群通知其他人还是很有必要的吧。。。
dcsite
2023-07-05 09:30:53 +08:00
你们 TL 确实是大煞笔…… 感觉完全是情绪化做事。

用邪恶一点的想法,生产分支不设限目的就是,可以权利最大化,随时可以整人……
nothingistrue
2023-07-05 09:32:38 +08:00
日常管理的规则是「规无约定则禁止」,并不是「规无禁止即可行」。刚入职场的小年轻容易把二者搞翻而碰壁。
tabris17
2023-07-05 09:33:59 +08:00
我觉得你工作心态有问题

“bug 拖一天生产上每日生成的文件名就错一天” 关自己什么事呢,管他呢
zhio
2023-07-05 09:37:48 +08:00
你的 tl, 逛不逛 V2EX 啊、
lingeo
2023-07-05 09:38:24 +08:00
git flow 不是靠那个 protect 来实现的,自己平时没有养成习惯,就当涨记性了吧。
你单独 commit 了 bugfix ,可以 revert 你的 bugfix 。
MuscleOf2016
2023-07-05 09:44:29 +08:00
不是自己主要负责的项目,就不要直接往 master push 了。应该走提合并请求的流程。
arnoldxiao
2023-07-05 09:44:48 +08:00
@tabris17 血的教训

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

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

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

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

© 2021 V2EX