ci: fixed the abnormal operation of ci files by kurisaW · Pull Request #10259 · RT-Thread/rt-thread
我再次在自己的仓库测试过关键性功能:
- 1.文件修改路径更新检测后自动生成审核名单
- 2.已经修改过的文件并且已经任命审核者后,第二次恢复该文件为初始状态(也就是不应该触发对应的审核者),与此也不应该生成对应评论
- 3.对于同一TAG,已经@过的审核者第二次删除@字符前缀(仅在第一次提及,避免因多次提及导致的message困扰问题)
- 4.对于不同审核TAG,有着同一审核者,同样满足第三点规则(假设:在TAG1中maintainer1已被提及,同时第二笔commit中触发了TAG2,恰好TAG2也是为maintainer1所任命,那么对于TAG1中的评论中,maintainer1的@前缀被去除;而TAG2则会再一次@提及,因为属于新文件审查,需要重新提及)
- 5.深路径文件匹配检测(递归检测满足MAINTAINERS的PATH规则)
- 6.LGTM审核同步(首次评论及审核者主动删除lgtm审核评论功能)
具体可参考下面这条github-bot的评论修改记录:
但是主线上的似乎仅支持第一次的maintainer ci效果运行,不清楚是否还有其他权限问题,又或者是因为触发条件为pull_request_target,对应的branch为master,也就是说,需要主线分支更新此规则后,才能在下一次的PR创建中才能被触发
@supperthomas @Rbb666 此PR作为一个修复补丁,可以先合并,然后重新运行PR的CI测试功能