同事直接往 master 推未经测试的代码

[justR]

最近新来个同事，代码写了就直接推到 master, 一堆 bug,说了也不听，遇到这种人怎么办

[ChefIsAwesome]

恕我直言，这都没人管的公司也就别在那待了吧。

[dfkjgklfdjg]

按规定办事咯。没有规定的话🤷‍♂️，得看你是他同事还是领导了。

[duange7X24]

谁出问题谁担责。如果你是他领导，屡教不改直接滚蛋；如果是同事，说了也不听，那就把火烧大

[TArysiyehua]

这个是你们开发规范的问题吧，跟你同事关系不大。
首先你们有项目文档吗？有没有说开发流程是怎么样的？比如必须先经过测试，通过了才能提交。代码覆盖于要多少，代码风格要怎么样？
其次，你们为什么没有对 master 进行限制与保护，新人可以直接往 master 推送本来就是很离谱的事情。
最后，你们的 review 机制是怎么样的？


这些东西你都不说，就说人家说了也不听。固然，一个对我自然要求较高的程序员，每次提交 的代码都是经过充分测试的，但是这个明显是考验每个程序员的自我修养。在很多公司个团队，靠自觉，本身就是个笑话

[EdmondGUO]

听起来不像什么正经公司

[alexHong]

他为什么能推上去。

[akira]

git 流程 做一下规范。。

[chendl111]

你们没有代码测试和代码复核的人员吗？流程上的问题不要怪人

[kirito41dd]

你们主分支能直接 push 也挺抽象的(

[Helsing]

找领导把 master 保护起来

[hello2090]

不是，等一下，一堆 bug 怎么了，谁的 code 没有 bug? 你 push 到 master 的没有 bug? 既然你们能直接 push 到 master 那他做的就没错啊。

[expkzb]

他推了，但还是应该有人负责审核和 merge 的吧

[anjingdexiaocai]

这种就别怪你同事了，代码流程就有问题。

[kisick]

公司的问题

[ivvei]

可以锁分支啊，为什么不锁。你们本来就没啥规范吧是不是？

[shimmerh]

这能推 master 本身说明公司不管控代码啊。。。这不是组长干的吗？

[joyhub2140]

master 属于保护分支，只有管理员才可以推送，其余组员一律只能推 merge request 。

[ytmsdy]

锁分支不就完了么。不允许直接 push 代码到 main 分支。多大点事情

[gosky]

小心根本不是正经公司，不可能树立规范
如果不幸是的，赶紧跑路

[iyaozhen]

我们 push 不了，不然新人来了不知道，很容易误操作呀