Repository navigation
ci: 组织邀请收窄到 members/ 个人介绍 PR - #13
Merged
Merged
Conversation
把邀请门槛从「任意 PR 合并即邀请」收窄为「PR 改动了 members/ 下的 个人介绍文件才邀请」,与 README/CONTRIBUTING 既有的「个人介绍 PR 才 发邀请、挑战与自由实验不作为邀请条件」规则对齐。members/README.md 与 members/example.md 属仓库脚手架,不计入。workflow_dispatch 手动 触发不受限制,保留补发入口。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
这次做了什么
把新人邀请门槛从「任意 PR 合并即邀请」收窄为「PR 改动了
members/下的个人介绍文件才邀请」。为什么
README.md/CONTRIBUTING.md原本就写明「个人介绍 PR 审核并合并后发送组织邀请」「挑战与自由实验是可选练习,不作为邀请条件」,之前的实现比文档宽松。all-membersteam 承载的是全员基础权限(目前挂了私有仓库codepaintstudio/x的 read),门槛应收紧。判定规则
改动文件匹配
^members/.+\\.md$,且排除仓库脚手架members/README.md、members/example.md。用历史 PR 回放验证:
members/AAA-pig.mdmembers/LQMN-CXK.mdlum-en.txt,members/lum-en.mdlum-en.txt1843128592.md(放错目录)如何验证
workflow_dispatch手动触发不受限制,仍是补发入口。members/,合并后应看到 workflow 以「跳过邀请」结束。