Identify if `push` event is for a pull request merge without additional API calls
还没有人认领这个 Issue。
评估
调研方向
首先阅读 issue 中描述的 push 事件处理程序和 pull request 事件处理程序。跟踪当前 API 调用如何确定推送的提交是否属于已合并的 pull request,以及何时创建 RSL 条目。确定事件数据是否可以替代 API 调用,并就由此产生的处理方式达成一致后,即视为完成。
由索引模型根据 Issue 内容生成。
描述
Currently, the push event handler creates an RSL entry for the reference in question. As GitHub sends a push event and a pull request event when a PR is merged, the push event handler uses the API to identify if the pushed commit is part of any pull requests that were merged. If so, an RSL entry is not created because the pull request handler takes care of it instead. This issue is to discuss if we can achieve this without the additional API call, especially since the API response may not indicate that the PR was merged (there may be a delay before the change in status is reflected in the API response).
- 主要语言
- Go
- 星标
- 5
- 派生
- 1
- 平均合并
- 8 小时 40 分钟
- 30 天内合并 PR
- 6
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
gittuf/github-app 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 45/100
gittuf/github-app#152 · 3 条评论 ·
-
beta testing github-app可能重新可做 @patzielinski 于 518 天前认领,目前没有进行中的 PR。 未关闭
gittuf/github-app#121 · 3 条评论 · 已指派 2 人 ·
-
难度 3/5 1-2 天 新手友好度 35/100
gittuf/github-app#108 ·
-
难度 4/5 3-5 天 新手友好度 30/100
gittuf/github-app#54 · 1 条评论 ·
查看 gittuf/github-app 的全部 Issue
相似的 Issue
-
enhancement
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
superplanehq/superplane#8604 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
documentation hacktoberfest priority: low skill: good first issue status: ready for dev
难度 2/5 半天 新手友好度 85/100
hyperledger-labs/fabric-smart-client#2142 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
numberly/vault-db-injector#107 ·
维护者通常 1 天内回复