这是一个创建于 62 天前的主题，其中的信息可能已经有所发展或是发生改变。
最近新来个同事，代码写了就直接推到 master, 一堆 bug,说了也不听，遇到这种人怎么办
 |
1
ChefIsAwesome 62 天前  16
恕我直言，这都没人管的公司也就别在那待了吧。
|
 |
2
dfkjgklfdjg 62 天前
按规定办事咯。没有规定的话🤷♂️，得看你是他同事还是领导了。
|
 |
3
duange7X24 62 天前
谁出问题谁担责。如果你是他领导，屡教不改直接滚蛋；如果是同事，说了也不听，那就把火烧大
|
 |
4
TArysiyehua 62 天前 8
这个是你们开发规范的问题吧，跟你同事关系不大。
首先你们有项目文档吗？有没有说开发流程是怎么样的？比如必须先经过测试，通过了才能提交。代码覆盖于要多少，代码风格要怎么样？ 其次，你们为什么没有对 master 进行限制与保护，新人可以直接往 master 推送本来就是很离谱的事情。 最后，你们的 review 机制是怎么样的？ 这些东西你都不说，就说人家说了也不听。固然，一个对我自然要求较高的程序员，每次提交 的代码都是经过充分测试的，但是这个明显是考验每个程序员的自我修养。在很多公司个团队，靠自觉，本身就是个笑话 |
 |
5
EdmondGUO 62 天前
听起来不像什么正经公司
|
 |
6
alexHong 62 天前 8
他为什么能推上去。
|
 |
7
akira 62 天前
git 流程 做一下规范。。
|
 |
8
chendl111 62 天前
你们没有代码测试和代码复核的人员吗？流程上的问题不要怪人
|
 |
9
kirito41dd 62 天前 via iPhone 3
你们主分支能直接 push 也挺抽象的(
|
 |
10
Helsing 62 天前 via iPhone
找领导把 master 保护起来
|
 |
11
hello2090 62 天前 1
不是，等一下，一堆 bug 怎么了，谁的 code 没有 bug? 你 push 到 master 的没有 bug? 既然你们能直接 push 到 master 那他做的就没错啊。
|
 |
12
expkzb 62 天前
他推了，但还是应该有人负责审核和 merge 的吧
|
 |
13
anjingdexiaocai 62 天前 via Android
这种就别怪你同事了，代码流程就有问题。
|
 |
14
kisick 62 天前 via iPhone
公司的问题
|
 |
15
ivvei 62 天前
可以锁分支啊，为什么不锁。你们本来就没啥规范吧是不是？
|
 |
16
shimmerh 62 天前 via iPhone
这能推 master 本身说明公司不管控代码啊。。。这不是组长干的吗？
|
 |
17
joyhub2140 62 天前
master 属于保护分支，只有管理员才可以推送，其余组员一律只能推 merge request 。
|
 |
18
ytmsdy 62 天前
锁分支不就完了么。不允许直接 push 代码到 main 分支。多大点事情
|
 |
19
gosky 62 天前
小心根本不是正经公司，不可能树立规范
如果不幸是的，赶紧跑路 |
 |
20
iyaozhen 62 天前
我们 push 不了，不然新人来了不知道，很容易误操作呀
|
 |
22
vitoliu 62 天前
我就经常往 master 推代码，因为我们是特性分支出包，所以 master 就是 dev~
|
 |
23
greatghoul 62 天前
啥公司管理这么混乱，pr review 都没有 😂
|
 |
24
NickX 62 天前
为什么 master 能随便 push ，收回权限使用 Merge Request 吧
|
 |
25
1647846714 62 天前
master 可以设置 push 权限的。
|
 |
26
iseki 62 天前
小公司吧，甚至一个仓库就一个人自己玩？那流程什么的有时候就多余了，自然是想怎么搞怎么搞，push -f 都会是家常便饭。
|
 |
27
dxk611 62 天前
这种人肯定有问题，但是最大问题在公司管理上
|
 |
28
ST0RMTR00PER 62 天前
为什么要给他权限？
|
 |
29
sarices 62 天前
不知道你的同事什么水平，但你们项目管理负责人应该负最大的责任
|
 |
30
phrack 62 天前 via iPhone 14
楼主暴露了同事的水平的同时，也暴露了自己的水平，领导的水平，还有公司的水平。
|
 |
31
warcraft1236 62 天前 1
你能相信我们几千人的全球业务的公司，master 分支其实和 dev 分支差不多，每天都有人往上边推不是足够稳定的代码，所以我经常遇到的问题就是，拉了最新的 master 分支的代码->编译报错->GG
|
|
32
warcraft1236 62 天前
@warcraft1236 几千人指的是我所在的大项目的组，不是整个公司
|
 |
33
SekiBetu 62 天前
出了问题有人负责就行了
|
 |
34
kenvix 62 天前
快跑
|
 |
35
Adelell 62 天前 via iPhone
真正的大神都是直接 push main ，只是人家没 bug 。
mastxx ，慎用，影响仕途。 |
 |
36
leaflxh 62 天前
小心离职后成为下一个 bilibili-master
|
 |
37
jayjoe 62 天前
没有 git 流程规范管理是相当可怕的一件事。。
|
 |
38
Mirage09 62 天前
branch protection 很重要
我自己也好几次手滑往 master branch push ，但是因为有保护都被 reject ，这才合理 |
 |
39
wm5d8b 61 天前 via Android
“培训班老师说了，软件开发就是写代码”🤪
新同事不都是这样讲的吗，测试是什么？培训班没教过 |
 |
40
allgy 61 天前
你们 master 是任何人都能推送的吗？
|
 |
41
manhere 61 天前
拿 git 当 ftp 用的公司不在少数
|
 |
42
xinyu391 61 天前
把 master 推送权限关了，
他只能推送分支，然后提交 merge 请求， |
 |
43
lilei2023 61 天前
小公司，没啥稀奇的，都是一个人搞，我这还有人直接在把 dev 当作 master 再用！ master 都落后 n 多版本了！
|
 |
44
felmoon 61 天前
master 咋能推上去的呢？
|
 |
45
jydeng 61 天前
权限太大了，能直接推 master
|
 |
46
huwenzhe 61 天前
他既然都能推了，还有什么问题么
|
 |
47
tairan2006 61 天前
master protect 一下不就行了
|
 |
48
changepll 61 天前
这有啥的. 我同事把 master,develop,还有一个功能分支来回 merge. 说多了还给你甩脸色. 我懒得说了.
有事就赖他身上. 挺好的 |
 |
49
xx6412223 61 天前
说实话，这和之前传言青岛啤酒有人往原材料里尿尿一样
一个懒人+一个草台班子 |
 |
50
deltaone 61 天前
用制度 (git setting) 来防止犯错。
|
 |
52
snoy 61 天前
我的规矩就是规矩，整些没用都是瞎扯，你是 leader 的情况下可以，其他的无所谓直接向上反映。
|
 |
53
351994 61 天前
能直接推 master 的也确实绷不住了
|
 |
54
MuscleOf2016 61 天前
如果这个库，和你没啥关系，随便他弄就行了。如果这个库和你有关系，就别让他瞎搞。
|
 |
55
bruce0 61 天前
像楼上说的, 锁一下 master 分支就行了
我们用 svn, 以前总有人不 fmt 代码就提交, 后来我写了一个脚本, 检查代码格式, 格式有问题 直接 拒绝提交,从此没有这个烦恼了 |
 |
56
ExplodingFKL 61 天前
上面不管那你也不管，只要把强制提交关闭就行，到时候别背锅的都找不到
|
 |
57
colorcat 61 天前
如果你们团队使用主线开发模型，那就是应该往 master 分支推送啊
|
 |
58
zhanshen1614 60 天前
任何人可以合并到主分支说明流程不规范，应锁定生产环境的分支，合并要走合并请求不能线下合并推到线上。弄个 git 流程不然太乱了。
|
|
59
zhanshen1614 60 天前
@jayjoe 非常可怕。我之前在一个项目组没有分支规范堆积数百上千个分支，有人看走眼删了一个没开发完的分支碰巧遇上重启任务导致整个重写。我给上级建议弄个 git 分支规范还被反驳：“你要关注业务产出！领导的决策要兼顾大多数团队，其他组的人都没意见为什么就你有意见？”。
|
 |
60
l4ever 60 天前
你自己不设置分支保护怪人家咯?
新同事可能他以前自己写一个项目, 就直接推 master 了呗, 你要么保护起来, 要么告诉他不要推呗 |
 |
61
Aruok 60 天前
还是项目流程不规范啊.
|
Select Language