Skip to content

🔧 ci: add stable required-check gate - #470

Merged
seakee merged 1 commit into
devfrom
ci/required-check-gate
Aug 1, 2026
Merged

🔧 ci: add stable required-check gate#470
seakee merged 1 commit into
devfrom
ci/required-check-gate

Conversation

@seakee

@seakee seakee commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

Add one stable aggregate check for branch protection. It preserves path-selective CI while preventing docs-only or skipped matrix jobs from leaving required check contexts permanently expected.

Scope

  • Frontend panel
  • Manager Server
  • CPA panel mode
  • Full Docker mode
  • Native packages / release
  • Docs / Wiki
  • CI / build / tooling

Changes

  • Add the always-run Required checks job after all path-selected PR jobs.
  • Treat successful and intentionally skipped checks as valid.
  • Fail the aggregate whenever an applicable prerequisite fails, is cancelled, or has no result.

User Impact

Contributors can merge docs-only and other narrow PRs after the applicable CI completes, while failed frontend, backend, native, or Docker checks still block protected branches.

Compatibility / Runtime Notes

  • CPA panel mode: N/A.
  • Manager Server mode: N/A.
  • Full Docker / native packages: CI gating only; no runtime behavior changes.

Data / Security Notes

No credentials, runtime configuration, database data, or release secrets are changed.

Risk / Rollback

Risk level: Low

Rollback notes:

Revert this workflow commit and restore the previous required-check configuration if the aggregate gate behaves unexpectedly.

Verification

  • Type check
  • Lint
  • Tests
  • Build
  • Manual UI check
  • Docs/link check
  • Not applicable, docs-only

Commands / evidence:

ruby -e 'require "yaml"; YAML.load_file(".github/workflows/pr-check.yml")'
git diff --check
bash -n bin/release/package-native.sh

Screenshots / Recordings

N/A — CI workflow only.

Docs

  • README / README_CN updated for user-visible capabilities
  • Matching docs manual and navigation updated
  • Demo fixtures, screenshots, and deep links reviewed
  • Release notes needed
  • Not needed — internal CI gate behavior only.

Docs decision:

No user-facing documentation changes are required.

Related

Refs #469

Aggregate the path-selected PR jobs into one required status check.\nAllow skipped jobs for unaffected paths while failing the aggregate on any failed or cancelled applicable check.\nThis keeps branch protection effective for docs-only and matrix-based pull requests.
@seakee
seakee merged commit 99e29d6 into dev Aug 1, 2026
10 checks passed
@seakee seakee mentioned this pull request Aug 1, 2026
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant