From 47c189f6d61c25a916930d5078944db0fcf8ac6c Mon Sep 17 00:00:00 2001 From: matteothebroketa <92643871+matteothebroketa@users.noreply.github.com> Date: Mon, 27 Jul 2026 14:09:54 +0200 Subject: [PATCH] Use unique aggregate check names --- .github/workflows/ci.yml | 2 +- .github/workflows/security.yml | 2 +- docs/CI_ARCHITECTURE.md | 4 ++-- docs/GITHUB_SETUP.md | 2 +- docs/RELEASING.md | 4 ++-- tools/validate_repository.py | 4 ++-- 6 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index f55d2ea..d3283d2 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -184,7 +184,7 @@ jobs: run: python tools/stress_atomicity.py --writers 12 --replacements 80 --payload-size 96000 required: - name: Required + name: CI required if: always() needs: [repository, quality, tests, coverage, package, stress] runs-on: ubuntu-latest diff --git a/.github/workflows/security.yml b/.github/workflows/security.yml index ae5bc9e..04f7b58 100644 --- a/.github/workflows/security.yml +++ b/.github/workflows/security.yml @@ -81,7 +81,7 @@ jobs: category: /language:${{ matrix.language }} required: - name: Required + name: Security required if: always() needs: [workflow-policy, dependency-review, codeql] runs-on: ubuntu-latest diff --git a/docs/CI_ARCHITECTURE.md b/docs/CI_ARCHITECTURE.md index 5623109..2d12ff4 100644 --- a/docs/CI_ARCHITECTURE.md +++ b/docs/CI_ARCHITECTURE.md @@ -21,7 +21,7 @@ Every push and pull request runs `.github/workflows/ci.yml`: contents and metadata, installs the wheel without an index or dependencies, and rebuilds an equivalent wheel from the source archive. 6. A concurrent reader/writer stress test runs on all three operating systems. -7. One stable `Required` job aggregates every result for branch protection. +7. One stable `CI required` job aggregates every result for branch protection. The test matrix installs the package before running tests. It therefore tests what packaging exposes rather than importing `src/` through an environment @@ -31,7 +31,7 @@ variable. `.github/workflows/security.yml` runs workflow-policy enforcement, dependency review for pull requests, and CodeQL analysis for both Python and GitHub Actions. -The workflow has a separate stable `Required` aggregate check. +The workflow has a separate stable `CI required` aggregate check. `.github/dependabot.yml` proposes reviewed updates for development tools and pinned GitHub Actions. Dependency updates never publish automatically. diff --git a/docs/GITHUB_SETUP.md b/docs/GITHUB_SETUP.md index 86af6a1..ddcd325 100644 --- a/docs/GITHUB_SETUP.md +++ b/docs/GITHUB_SETUP.md @@ -44,7 +44,7 @@ Create a branch ruleset targeting `main` with these controls: - stale approvals are dismissed after new commits; - all review conversations must be resolved; - the branch must be current before merge; -- required checks are `CI / Required` and `Security / Required`; +- required checks are `CI / CI required` and `Security / Security required`; - force pushes and branch deletion are blocked; - linear history is required; - administrators should not bypass the rules during ordinary work. diff --git a/docs/RELEASING.md b/docs/RELEASING.md index ec1b23c..f4c3de0 100644 --- a/docs/RELEASING.md +++ b/docs/RELEASING.md @@ -8,8 +8,8 @@ upload production distributions. 1. Merge all intended changes through `main`. 2. Update the project version, package `__version__`, and dated changelog entry. 3. Run `python tools/ci.py all` where development tools are installed. -4. Merge the release-preparation pull request after `CI / Required` and - `Security / Required` pass. +4. Merge the release-preparation pull request after `CI / CI required` and + `Security / Security required` pass. 5. Confirm the latest scheduled extended validation has no unexplained failure. ## Publish diff --git a/tools/validate_repository.py b/tools/validate_repository.py index bada519..aac6b7a 100644 --- a/tools/validate_repository.py +++ b/tools/validate_repository.py @@ -121,7 +121,7 @@ def _validate_workflow_architecture(errors: list[str]) -> None: "python tools/typecheck_consumers.py", "python tools/rebuild_from_sdist.py dist", "python tools/stress_atomicity.py", - "name: Required", + "name: CI required", ), "security.yml": ( "language: [python, actions]", @@ -129,7 +129,7 @@ def _validate_workflow_architecture(errors: list[str]) -> None: "actions/dependency-review-action@", "github/codeql-action/init@", "github/codeql-action/analyze@", - "name: Required", + "name: Security required", ), "extended.yml": ( "python tools/repeat_tests.py --runs 15",