gitlab ci 如何在提交到 master 分支前进行验证，通过验证才能 merge 或 push

[css3]

请教下各位大佬：

• 写了个验证脚本，放到代码工程中，并且将执行任务加到 ci stage 里边了
• 但现有的 ci 执行顺序是大家先 push 或 merge 然后才进入 stage 自动流程，然后才走的脚本验证流程，所以这个验证功能并没有什么用，验证未通过时，也还是合到 master 上了
• 怎么才能在 merge 或 push 前验证出来结果呢？验证不过，就拒绝提交 master

[nightwitch]

禁止提交到 master，要提交到 master 发 pull request 。发 pull request 的时候自动触发 CI 测试，CI 测试不过不允许合并

[zmxnv123]

git hook?

[Rwing]

@nightwitch 这个是正解

[Sapp]

coommit 可以添加钩子啊，前端就有个叫哈士奇的，不通过 commit 都没办法提交

[phy25]

比较优雅的办法，自动提交 MR：

https://docs.gitlab.com/ee/user/project/merge_requests/merge_when_pipeline_succeeds.html

https://docs.gitlab.com/ee/user/project/push_options.html

git push -o merge_request.create -o merge_request.target=my-target-branch -o merge_request.merge_when_pipeline_succeeds

[janxin]

禁止 master 分支直接提交

设置 Protected Branches

合并 MR 需要在 pipelines 执行成功

设置 Only allow merge requests to be merged if the pipeline succeeds

[maltoze]

https://github.com/pre-commit/pre-commit

[gen900]

gitlab 里 没有通过 pipeline 的 merge request 不让合并的。就是你 push 后就在做 ci 了如果失败 mr 不让合并

[wshcdr]

同意 MR 的做法

[otakustay]

按照楼主的要求，其实你一开始就应该用 gerrit 而不是 gitlab

[dddd1919]

提 git 的人不按规范执行，总有方法绕过你的设置，不统一提交规范，其他的就别扯了
用 git 靠的是约定，不是靠各种限制，如果大家都不遵守约定，那就找人专门来干这事，或者换个工具吧。
统一 git flow，再用 ci 才能达到比较好的效果，比如配置 ci 在 feature 上执行，禁用 remote push master，合并 mr 前要 ci 通过，ci 同时配置 build 和 run test 等等

[icylogic]

我们 master/develop 任何人都不能推的，所有更改都要通过其他分支提 mr，mr 只能由 reviewer 点合并，如果 ci pipeline 挂掉或者有 conflict，合并按钮都是灰的没法点（可以选项里改）

[wxsm]

我通过 gitlab 的 webhook 和 api 自己实现了一套评审系统，详见： https://wxsm.space/posts/2020-09-23-gitlab-ce-code-review-bot.html

[dengxuejiu]

亲测可用，在 merge 之前做代码规范检查等工作还是挺好用的