Repository ruleset rule type `code_quality` is not modelled
@etifontaine 已经在做这个了。
开始于 2026年9月4日。
评估
这个 Issue 还没有评估数据。
描述
Summary
The repository ruleset rule type code_quality is not modelled by RepositoryRule / the ruleset rule types, so it cannot be read or written through this library. It corresponds to the "Require code quality results" checkbox on a branch ruleset, which reached general availability on 2026-07-20.
API shape
GET /repos/{owner}/{repo}/rulesets/{id} returns it as a rule alongside the existing types:
{
"id": 16352232,
"name": "Project branch",
"target": "branch",
"enforcement": "active",
"rules": [
{ "type": "code_quality", "parameters": { "severity": "errors" } },
{ "type": "copilot_code_review", "parameters": { "review_on_push": true, "review_draft_pull_requests": false } }
]
}
severity is the lowest severity level at which code quality results block a merge. I have only observed "errors" on the wire, so the full accepted set is worth confirming against the docs before fixing an enum — note the existing code_scanning rule's security_alerts_threshold already uses "errors" / "errors_and_warnings" style values in rules.go, which may or may not be the same vocabulary here.
Why it matters beyond a missing field
PUT /repos/{owner}/{repo}/rulesets/{id} replaces the entire rules array. Because the rule cannot be represented, any consumer that round-trips a ruleset — read, modify one unrelated attribute, write back — will silently drop code_quality and remove a merge gate, with nothing in the request or response to indicate it happened.
That is not hypothetical. integrations/terraform-provider-github builds its github_repository_ruleset resource on this library, and a ruleset carrying this rule imports cleanly, plans cleanly, and would lose the rule on the first apply that touched any other attribute of the resource. We ended up dropping the affected ruleset from Terraform management entirely rather than hold that risk. Modelling the type here unblocks the provider side (there is no provider-side workaround, since there is nothing to serialise).
Environment
- go-github: absent as of current
master—github/rules.gomodelscopilot_code_review,required_code_scanning,required_status_checkset al., and contains no occurrence ofcode_qualityorCodeQuality - Observed against: GitHub.com, private repository in an organisation on the Team plan
Willing to contribute
Happy to open a PR adding the rule type if a maintainer confirms the preferred shape — in particular whether severity should be a plain string or a typed constant set.
- 主要语言
- Go
- 星标
- 11.3k
- 派生
- 2.5k
- 平均合并
- 1 天 16 小时
- 30 天内合并 PR
- 51
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
google/go-github 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 3/5 1-2 天 新手友好度 68/100
相似的 Issue
-
bug github_actions
难度 2/5 1-3 小时 新手友好度 75/100
registrystack/registry-stack#1393 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
JakeChampion/lang#10213 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
oasisprotocol/oasis-sdk#2523 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 70/100