Pygithub: 添加对高级分支保护 API 的支持

创建于 2017-06-30 · 15评论 · 资料来源: PyGithub/PyGithub

似乎以下 API 存在于 github v3（作为 beta 版？）中，但尚未反映在此库中。

https://developer.github.com/v3/repos/branches/#get -pull-request-review-enforcement-of-protected-branch
https://developer.github.com/v3/repos/branches/#update -pull-request-review-enforcement-of-protected-branch
https://developer.github.com/v3/repos/branches/#get -admin-enforcement-of-protected-branch
https://developer.github.com/v3/repos/branches/#get -restrictions-of-protected-branch
和其他几个......

完全支持这个 API 会很棒（特别是因为它允许控制 PR 审查以及需要多少和由谁）。

feature request high priority

资料来源

harlowja

👍5

最有用的评论

这实际上是一个大混乱。 Repository.protect_branch() 调用了一个可能已被删除的端点，因为 GitHub 博客说这将是分支保护毕业的时候。我一直想知道为什么 API 文档和代码完全不一致。

我将添加一个新方法 Repository.edit_branch_protection() 以使用新数据结构调用新端点，并且我很想同时删除 Repository.protect_branch() 被破坏。

s-t-e-v-e-n-k 于 2018-04-30

👍3

所有15条评论

@harlowja我在https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca中做了一点

对我来说至少审核我们的分支保护就足够了。 YMMV。

a13m 于 2017-07-05

是的，将类似的东西合并（并发布）真是太好了；这个库似乎落后于实际的 API 和/或功能，这让我有点难过：-/

harlowja 于 2017-09-12

特别喜欢这个，因为我一直在努力确保我的组的所有 github 配置都是 yaml 格式，而不是隐藏在 github UI 中，然后有一个程序可以解析所述 yaml 格式并调整各种 github 存储库。这在我的组存储库中创建了连续性（和健全性），并且能够以编程方式调整审阅者保护在逻辑上是团队想要做的事情（以便可以通过类似的 yaml 格式控制和管理审阅者列表）。

harlowja 于 2017-09-12

干得好@a13m！
同时： https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/。

看起来我们被困住了，直到有人集中精力解决这个问题。

您现在需要在调用更新分支保护端点时传递 required_pull_request_reviews 对象。在预览期间传递此对象是可选的。

tomgoren 于 2017-10-03

我也很想看到这个。请公关❤️

sfdye 于 2018-04-06

我将添加一个新方法 Repository.edit_branch_protection() 以使用新数据结构调用新端点，并且我很想同时删除 Repository.protect_branch() 被破坏。

s-t-e-v-e-n-k 于 2018-04-30

👍3

大家好，我的补丁还没有完成，但我认为有早期反馈会很好^

JPWKU 于 2018-05-10

我也一直在研究这个问题，以不同的方式处理它——通过将方法添加到 Branch，并删除 Repository.protect_branch()

s-t-e-v-e-n-k 于 2018-05-11

@stevenk 啊，我现在明白了。嗯，不想重复工作。我的意图是保持库 api 相同。你想怎么做？

JPWKU 于 2018-05-11

@JPWKU对不起，我应该更清楚地传达我正在积极开展这项工作。 Repository.protect_branch() 方法确实与 GitHub API 匹配，直到他们在新的分支保护 API 毕业时将其删除。

我当前的补丁实现了 GitHub 在https://developer.github.com/v3/repos/branches/上显示的分支保护 API，因此它相当大。在我提出这个之前，我有 3 个左右的测试用例要完成编写。

s-t-e-v-e-n-k 于 2018-05-12

@stevenk 不用担心，因为你已经在这个领域工作了，我会关闭我的 PR 并评论/帮助你的。

JPWKU 于 2018-05-12

@stevenk 你打算很快合并吗？我需要这个功能。

prburgu 于 2018-05-22

@prburgu合并时不取决于我。 @JPWKU你什么时候有时间继续审查附加的 PR？

s-t-e-v-e-n-k 于 2018-05-22

@prburgu @stevenk 我将尝试在今天或今晚抽出时间来完成审查。

JPWKU 于 2018-05-22

👍2

这事有进一步更新吗？

rt-alex 于 2018-07-26

👍2

此页面是否有帮助？

0 / 5 - 0 等级

Pygithub: 添加对高级分支保护 API 的支持

最有用的评论

所有15条评论

相关问题