对提交到 gitlab 的代码 message 进行验证以符合 Angular 规范
gouchaoer · 2020-06-22 14:01:28 +08:00 · 3208 次点击这是一个创建于 1652 天前的主题，其中的信息可能已经有所发展或是发生改变。
现在团队采用 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，看相等就跳过验证
 |
1
flyingfz 2020-06-22 14:19:32 +08:00
搜一下 git cz
|
|
2
flyingfz 2020-06-22 14:19:37 +08:00
|
 |
3
gouchaoer OP @flyingfz 我是在服务器端验证的，不是客户端，客户端没法分发这个工具。。。。还有我除了要符合 Angular 规范，还有别的定制功能，比如和项目管理之间联动。。。这个 cz 依赖太复杂，我生成 CHANGELOG 调研了一下，选了 golang 一个，直接编译成二进制无依赖
|
 |
4
yuxizhe 2020-06-22 19:27:03 +08:00
gitlab ci 配置个 commitlint 就行了，不要瞎造轮子
https://github.com/conventional-changelog/commitlint |
 |
5
SoloCompany 2020-06-22 22:50:35 +08:00
如果没有 MR, 仅仅靠 Hook 没有多少意义
如果有 MR, 那干嘛还要 Hook? 有意义的替代品应该是 approval robot |
|
6
gouchaoer OP @SoloCompany 如果我说要去过认证，基础设施里 message 规范除了这个之外还需要一些特殊的与项目管理的联动呢？你放出来的工具早调研过了，实际项目比你想象的复杂的多。。。你只想到在 ci 里配 lint，实际操作中那么多人不是每个项目都有 ci 好吧？如果是用 repo 管理的许多 git 仓库呢（实际生成 CHANGELOG 的时候需要每个仓库都做一次）？
@yuxizhe 说到底我只是简单的对 git commit message 做一个要求，我认为这是很 easy 的工作，我几行正则就搞定了 |
Select Language