我发现普通开发者没有权限向保护分支/只读分支 merge pr,这很奇怪,review 通过之后应该有权限自己做这个事情呀
省去管理员接受 pr 的操作
github
保护分支只读分支只能管理员了可以merge 进去,PR的合并权限是对分支有push权限才有合并权限,无论是否review,因为review 本来也是个options,不能作为是否可以合并的充分条件哦,充分条件只能是已是否有push 权限
此处可能存在不合适展示的内容,页面不予展示。您可通过相关编辑功能自查并修改。
如您确认内容无涉及 不当用语 / 纯广告导流 / 暴力 / 低俗色情 / 侵权 / 盗版 / 虚假 / 无价值内容或违法国家有关法律法规的内容,可点击提交进行申诉,我们将尽快为您处理。
@Yashin 感觉有些不太方便,希望可以分开控制成员对默认分支的 push 、合并 PR 权限。谢谢。
@Helson 如果默认分支不是保护分支或者只读分支的话成员也是可以合并的呢
@Yashin 那这样成员就可以 push 了,我不希望开放这个操作,不安全。
@Helson 是的,那么不希望成员可以push也不希望成员可以随意合并pr才对,否则岂不是可以通过PR的形式来绕过push了?
@Yashin 简单来说,我希望大家都不能往默认分支 push,必须通过 PR ,而 PR 只要通过管理员审核就可以自己合并..不管他有没有 push 权限,我觉得合并 PR 和 push 权限应该是分开的。
不过问题不大,审核通过后再点一下合并就是了。
登录 后才可以发表评论