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

[breadykidliu]

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

[TestFlight]

常在河边走，哪有不湿鞋。某一次 push 之后导致线上异常，造成资损进行复盘的时候，你就知道了，你是主要责任人，TL 是次要责任人

[Felldeadbird]

为了楼主好，以后 push 前通知对应的人。

楼主估计工作被骂的少。

[xFrye]

「挺生气的，关于下属不事先沟通私下合并代码的这件事」

[Belmode]

这也不完全是你的错，工作上，遇事最好还是多沟通为好，千万不能相当然，不然会吃大亏的。需要做什么事，最好以邮件、会议、聊天群 at 所有人等方式，知会相关人员留痕。
不要听一些二极管的发言。

[jiangzm]

同样作为 TL ， 我一直在组内强调的是 发布上线的代码需要经过 CR （可以组员之间），任何变更内容都要通过测试验证。上面看似是要求其实也是责任分摊，出了问题不会一个人担着

[kumiko]

笑死，看来楼主没背过锅。多背几次有不会这么幼稚了

[shaozelin030405]

别。。。别乱往生产分支合东西，只合说好的东西

[lincanbin]

各打五十大板，你领导有管理责任，没有用权限管控等手段来限制这种事情的发生。
你的话则是代码合并上线没有及时周知。

[janus77]

首先领导是不会改的，该改的最终还得是你
其次你的理由在我看来很可笑，早一点修复是更好，但是有谁规定一定要早一点修复了吗？相反，还真有人规定过不能随便合分支。另外晚一点上线又怎么样了呢？是会被骂还是会扣钱吗？
很现实的问题。op 就是没搞清技术和管理的分界。

[Desiree]

@kalman03 这不是效率问题，是安全问题，生产分支都不保护，那 Git 管理有啥意义呢

[kalman03]

@Desiree 就看怎么理解“约定”与“约束”，在一个高素质的开发团队，每个人都是自驱的，约定总是能运行的很好。就好比，有些公司对打卡没有要求，但实际上每个人都能自行约束自己，产出远远高于打卡的公司。

[kalman03]

@Desiree 本来不打卡的公司运行的很好，突然一天来了一个楼主这样的员工，这样就打破了约定。哈哈

[kalman03]

@Desiree 那么是跟 OP 谈谈上班摸鱼的事情，还是开启强制打卡模式呢？我觉得这个其实是一个道理，ok ，温习下敏捷宣言

https://www.leangoo.com/wp-content/uploads/2018/07/leangoo%E6%95%8F%E6%8D%B7%E5%AE%A3%E8%A8%80.jpg

[fresco]

你想想为啥别人都没这个问题？

[xu45525584]

事情怎么样先不说，不过你这性格在职场不好混的，混社会还是讲人情的
领导处不好，基本可以走人了，因为后面有好事也没你的份

[houzhiqiang]

没有 code review 吗？不创建 pr 或者叫 mr 吗？

[chonphile1]

乱麻一团，赶紧离职吧，从头到尾都是错。
1.代码提交都没有合理的分支规划
2.任何人都可以随意 push ？
3.无测试、不审核的？
4.你们这是能有多自信？

[G2bN4dbX9J3ncp0r]

多沟通

[realpg]

OP 的问题
无论沟通还是做事都有问题 全面问题

[sampeng]

没过测试，也没和任何人，包括 leader 说代码合并到 master 的。被喷没任何问题。