对提交到 gitlab 的代码 message 进行验证以符合 Angular 规范

2020-06-22 14:01:28 +08:00
 gouchaoer

现在团队采用 Angular 代码规范,需要对提交到 gitlab 的代码进行验证,由于 gitlab 提供了 hooks 机制,所以做起来还是毕竟方便的,这里写一些我怎么做的

首先 Angular 代码规范参考: http://www.ruanyifeng.com/blog/2016/01/commit_message_change_log.html

gitlab 的 hook 机制参考: https://docs.gitlab.com/ee/administration/server_hooks.html

也就是说我们可以在 pre-receive 、update 和 post-receive 这 3 个阶段来对 commit message 进行验证,我之前写的的在 update 中验证的,你可以选你熟悉的脚本语言来写。在 gitlab 调用 update 脚本的时候会传入更新的 2 个 commit 的 hash ID,然后在这里可以调用 git 来解析出有哪些 commit 需要验证(你最好找个别人写好的解析 git commit message 的库),如果验证通过就返回 0,否则返回非 0,标准输出可以在 git push 的命令行中看到。需要注意的时候当 push 的为 tag 或者新的分支的时候,hash ID 可能为“0000000000000000000000000000000000000000”,还有就是“Merge branch”的需要考虑边界情况,大部分都是内容处理没啥可说的。如果你们还有别的要求,可以自己加。

Angular 规范正则可以写成:

//匹配 Angular 规范关键字

$res = preg_match("#^\s*(feat|fix|docs|style|refactor|test|chore)(\(.+\))?\:.+#", $message, $matches);
        if(!$res){
            $msg = "your commit message format is wrong, see:'http://www.ruanyifeng.com/blog/2016/01/commit_message_change_log.html', message:{$message}";
            if($NEED_VERIFY){
                log_file($msg, true);
                exit(1);
            }else{
                log_file($msg);
            }
        }

最后就是,在某些情况下我们需要 skip 掉验证,但是 hook 没有提供额外的参数,这里只有手动去改了,搜了一下 gitlab 用了 git push -o 的参数,在 pre-receive 下,加上这行:

refs = $stdin.read

require_relative '../lib/hooks_utils'
push_options = HooksUtils.get_push_options
if push_options[0]=="skip"
    aFile = File.new("/tmp/gitlab-skip", "w")
    if aFile
        content = aFile.syswrite(refs)
    else
        puts "Unable to open /tmp/gitlab-skip"
    end
end

然后在 update 脚本里检查的时候去 /tmp/gitlab-skip 读取 commit 的 hash,看相等就跳过验证

3170 次点击
所在节点    程序员
6 条回复
flyingfz
2020-06-22 14:19:32 +08:00
搜一下 git cz
flyingfz
2020-06-22 14:19:37 +08:00
gouchaoer
2020-06-22 15:58:09 +08:00
@flyingfz 我是在服务器端验证的,不是客户端,客户端没法分发这个工具。。。。还有我除了要符合 Angular 规范,还有别的定制功能,比如和项目管理之间联动。。。这个 cz 依赖太复杂,我生成 CHANGELOG 调研了一下,选了 golang 一个,直接编译成二进制无依赖
yuxizhe
2020-06-22 19:27:03 +08:00
gitlab ci 配置个 commitlint 就行了,不要瞎造轮子
https://github.com/conventional-changelog/commitlint
SoloCompany
2020-06-22 22:50:35 +08:00
如果没有 MR, 仅仅靠 Hook 没有多少意义
如果有 MR, 那干嘛还要 Hook? 有意义的替代品应该是 approval robot
gouchaoer
2020-06-28 10:38:00 +08:00
@SoloCompany 如果我说要去过认证,基础设施里 message 规范除了这个之外还需要一些特殊的与项目管理的联动呢?你放出来的工具早调研过了,实际项目比你想象的复杂的多。。。你只想到在 ci 里配 lint,实际操作中那么多人不是每个项目都有 ci 好吧?如果是用 repo 管理的许多 git 仓库呢(实际生成 CHANGELOG 的时候需要每个仓库都做一次)?
@yuxizhe

说到底我只是简单的对 git commit message 做一个要求,我认为这是很 easy 的工作,我几行正则就搞定了

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

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

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

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

© 2021 V2EX