Skip to content

ci: remove legacy aggregate gates #455

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
qiyi71w merged 2 commits into from
Sep 11, 2026
Merged

ci: remove legacy aggregate gates #455

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8dfe02b
ci: remove legacy aggregate gates
qiyi71w Sep 11, 2026
6caf46b
chore: sync main for CI gate cleanup
qiyi71w Sep 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 0 additions & 29 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,32 +154,3 @@ jobs:
run: |
printf '%s\n' "repository-checks=$REPOSITORY" "script-tests=$SCRIPTS" "windows-script-tests=$WINDOWS_SCRIPTS" "java-linux=$JAVA_LINUX" "java-windows=$JAVA_WINDOWS"
[[ "$REPOSITORY" == success && "$SCRIPTS" == success && "$WINDOWS_SCRIPTS" == success && "$JAVA_LINUX" == success && "$JAVA_WINDOWS" == success ]]

build:
if: ${{ always() }}
needs: [repository-checks, script-tests, java-linux]
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- name: Require Linux execution jobs
env:
REPOSITORY: ${{ needs.repository-checks.result }}
SCRIPTS: ${{ needs.script-tests.result }}
JAVA_LINUX: ${{ needs.java-linux.result }}
run: |
printf '%s\n' "repository-checks=$REPOSITORY" "script-tests=$SCRIPTS" "java-linux=$JAVA_LINUX"
[[ "$REPOSITORY" == success && "$SCRIPTS" == success && "$JAVA_LINUX" == success ]]

windows-script-validation:
if: ${{ always() }}
needs: [windows-script-tests, java-windows]
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- name: Require Windows execution jobs
env:
WINDOWS_SCRIPTS: ${{ needs.windows-script-tests.result }}
JAVA_WINDOWS: ${{ needs.java-windows.result }}
run: |
printf '%s\n' "windows-script-tests=$WINDOWS_SCRIPTS" "java-windows=$JAVA_WINDOWS"
[[ "$WINDOWS_SCRIPTS" == success && "$JAVA_WINDOWS" == success ]]
11 changes: 5 additions & 6 deletions docs/DEVELOPMENT.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,13 +103,12 @@ Windows 脚本 job 依次运行 `windows/repository` 与 `windows/scripts`，摘
两平台 Java job 保留全量测试、`LoggingProviderSmokeIT`、shaded JAR 和 JaCoCo；
验证成功但 coverage artifact 缺失仍失败。各组失败时仍尝试上传本组摘要。

`ci-required` 直接要求五项全部成功。迁移期间 `build` 汇总两个 Portable 非 Java job
和 `java-linux`；`windows-script-validation` 汇总两个 Windows job。三个汇总器均拒绝
失败、取消、跳过、缺失或未知结果。发布仍仅接受目标 SHA 的 `ci.yml` push 成功运行。
`ci-required` 是唯一汇总门禁，直接要求五项全部成功，拒绝失败、取消、跳过、缺失或
未知结果。发布仍仅接受目标 SHA 的完整 `ci.yml` push 成功运行。

首个 PR 合并且对应 main push CI 成功后，请 wimi321 将 main required checks 替换为
GitHub Actions 来源的 `ci-required`，其他保护设置不变。确认设置生效后，后续 PR
才删除两个旧汇总门禁；当前 PR 不需要管理员预先修改保护。
main 分支保护要求 GitHub Actions 来源的 `ci-required`。回退 workflow 时必须保留
可运行且真实验证五项执行结果的 `ci-required`；不能直接回退到缺少该检查的版本。
如需调整 required checks，由维护者协调保护设置与 workflow，始终保留有效合并门禁。

本地预检用于在推送前尽早发现问题，不能代替受保护分支上的干净 Windows 和
Ubuntu runner，也不能代替 macOS 签名、公证与多平台发布资产审计。
Expand Down
Loading