ci: merge queue runs the full main lane even for docs-only PRs (scope does not classify merge_group)
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 55/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- c, github-actions
- 领域
- build-system, ci-cd
调研方向
Look at the GitHub Actions workflow file that defines the 'scope' classification, likely in .github/workflows/. Understand how it currently checks for docs-only changes on pull_request events. Then examine the merge_group event context to see how the diff can be obtained against its base. The goal is to modify the scope logic to also classify merge_group candidates, so docs-only merges skip heavy lanes like macos-latest. Test by simulating a merge_group run or checking the queue's behavior after changes.
由索引模型根据 Issue 内容生成。
描述
Found while building the merge queue (#1264, PR #1273). It is out of that PR's scope, so it is filed here rather than folded in.
With the merge queue, every queued PR runs the FULL main lane on merge_group, including docs-only PRs. scope only classifies docs-only changes on pull_request, not on merge_group. The queue's wall time is set by macos / macos-latest running the full suite plus [99i]: about 35 min on main, against 16 on a PR (measured by the #1273 round-1 critic).
Decide: either teach scope to classify the queue candidate's diff against its base on merge_group (so docs-only merges skip the heavy lanes and still report the required checks as success), or accept the cost, since merges are a few a day. Measure the queue wall time over the first week before choosing.
Done when
- The owner records a decision here: (a)
scopeclassifies the queue candidate's diff againstmerge_group.base_shaonmerge_group, or (b) the cost is accepted. The decision cites the queue wall-time measurement fromgh run list -R InauguralSystems/EigenScript --event merge_group --workflow ci.yml --limit 50 --json createdAt,updatedAt. That measurement on 2026-09-27 covered 37 runs from 2026-09-23 to 09-27. The last 15 took 11–23 min, not the 35 min the issue cites: macos-15-intel moved to nightly and the [99i] audit became a grep in #1308. - If (a): a docs-only PR (only
*.mdfiles) goes through the queue withscopeloggingcode=falseon itsmerge_grouprun, and every check in.github/required-checks.txtreportssuccess. The run is linked. - If (a), planted fault: a queue candidate that touches one non-
.mdfile getscode=trueand runs the full lane. The run or a local replay of thedetectstep on that diff is recorded in the PR. - If (b): the issue is closed with the measurement above and the one-line reason.
- 主要语言
- C
- 星标
- 3
- 派生
- 7
- 平均合并
- 4 小时 7 分钟
- 30 天内合并 PR
- 112
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 提供 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
InauguralSystems/EigenScript 的其他 Issue
-
area:embed kind:silent-wrong
难度 2/5 1-3 小时 新手友好度 78/100
InauguralSystems/EigenScript#1387 ·
维护者通常 1 天内回复
-
area:stdlib kind:silent-wrong
难度 2/5 1-3 小时 新手友好度 86/100
InauguralSystems/EigenScript#1378 ·
维护者通常 1 天内回复
-
area:gates kind:gate-defect
难度 2/5 1-3 小时 新手友好度 78/100
InauguralSystems/EigenScript#1374 ·
维护者通常 1 天内回复
-
Error carets pad multi-byte UTF-8 byte-for-byte, so the ^ lands right of the token on a terminal未关闭area:lint-tooling kind:silent-wrong
难度 2/5 1-3 小时 新手友好度 84/100
InauguralSystems/EigenScript#1373 ·
维护者通常 1 天内回复
-
area:gates kind:docs-drift
难度 1/5 1 小时以内 新手友好度 88/100
InauguralSystems/EigenScript#1372 ·
维护者通常 1 天内回复
查看 InauguralSystems/EigenScript 的全部 Issue
相似的 Issue
-
feature request
难度 1/5 1 小时以内 新手友好度 86/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
FujiNetWIFI/fujinet-firmware#1730 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
-
难度 1/5 1 小时以内 新手友好度 92/100
-
category:port-update
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 2 天内回复