nodejs / nodejs/node-core-utils
Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval
未关闭
还没有人认领这个 Issue。
- 主要语言
- JavaScript
- 星标
- 313
- 派生
- 137
- 平均合并
- 2 天 8 小时
- 30 天内合并 PR
- 24
描述
IIRC collaborators are able to manually trigger CI through Jenkins without needing 1 approval on the PR, maybe it makes sense for the collaborators to use request-ci label on their own or other collaborators's PR without at least 1 approval restriction.
https://github.com/nodejs/node-core-utils/blob/a68af4b7744e8416bfc0f062570688b67d5e10dc/lib/pr_checker.js#L560 just an idea - could we check either the PR author is a current collaborator?
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
从 lib/pr_checker.js 第 560 行附近开始，跟踪 request-ci label 使用的批准限制。检查 collaborator 状态和 pull request 作者身份是如何表示的，然后验证相关的 CI-label 检查。完成标准是：符合条件的 collaborator 可以在自己的或另一位 collaborator 的 pull request 上使用 request-ci，而无需一次批准。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript
- 领域
- authorization, ci-cd
- Issue 类型
- 功能
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 56/100