mirror of
https://github.com/gradle/actions.git
synced 2026-06-12 14:40:39 +00:00
Daz DeBoer
8b6cdb5f58
Prepares CI so a small, stable set of **required status checks** can be enabled (which in turn unlocks auto-merge), instead of having to list every fanned-out matrix job. GitHub required checks match by exact name — no wildcards — so this reduces the surface to a handful of high-level checks. ## Changes - **`ci-integ-test.yml`**: add an aggregate gate job `integ-test-success` that `needs:` all four top-level jobs (the three suite jobs each wrap a reusable workflow that fans out into many nested checks) and fails if any did not succeed. `if: always()` ensures it reports even when a dependency fails. This collapses dozens of nested integ-test checks into a single requireable check. - **`ci-init-script-check.yml`**: remove the workflow-level `pull_request.paths` filter so the workflow runs on every PR and always reports a status check (previously it was absent on most PRs, which would deadlock a required check). Relevant-change detection moves into the job via `tj-actions/changed-files` (same pinned action already used by `ci-check-no-dist-update.yml`). On a PR the Java/Gradle/test steps run only when init-script files changed; otherwise the job is a fast no-op that still succeeds. Push and `workflow_dispatch` runs execute fully as before. ## Suggested required-check set (all run on every PR, none can deadlock) - `CI-check-and-unit-test / check-format-and-unit-test` - `ci-validate-typings.yml / validate-typings` - `CI-validate-wrappers / validation` - `CI-codeql / Analyze (javascript-typescript)` - `CI-integ-test / integ-test-success` - `CI-init-script-check / test-init-scripts` `ci-check-no-dist-update` is intentionally **omitted** — it only runs on `dist/**` edits and is designed to fail, so it shouldn't be a required gate. > Confirm the exact check names from the list GitHub shows after this branch runs once. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
81 lines
2.4 KiB
YAML
81 lines
2.4 KiB
YAML
name: CI-integ-test
|
|
|
|
on:
|
|
workflow_dispatch:
|
|
pull_request:
|
|
push:
|
|
branches:
|
|
- 'main'
|
|
- 'release/**'
|
|
- 'dev/**' # Allow running tests on dev branches without a PR
|
|
paths-ignore:
|
|
- 'dist/**'
|
|
|
|
permissions:
|
|
contents: read
|
|
|
|
jobs:
|
|
build-distribution:
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Checkout sources
|
|
uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
|
|
- name: Build and upload distribution
|
|
if: ${{ needs.determine-suite.outputs.suite != 'full' }}
|
|
uses: ./.github/actions/build-dist
|
|
|
|
caching-integ-tests:
|
|
needs: build-distribution
|
|
uses: ./.github/workflows/suite-integ-test-caching.yml
|
|
concurrency:
|
|
group: CI-integ-test-caching-${{ github.ref }}
|
|
cancel-in-progress: false
|
|
with:
|
|
skip-dist: false
|
|
secrets: inherit
|
|
|
|
other-integ-tests:
|
|
permissions:
|
|
contents: write
|
|
needs: caching-integ-tests
|
|
uses: ./.github/workflows/suite-integ-test-other.yml
|
|
concurrency:
|
|
group: CI-integ-test-other-${{ github.ref }}
|
|
cancel-in-progress: false
|
|
with:
|
|
skip-dist: false
|
|
secrets: inherit
|
|
|
|
dependency-submission-integ-tests:
|
|
permissions:
|
|
contents: write
|
|
needs: other-integ-tests
|
|
uses: ./.github/workflows/suite-integ-test-dependency-submission.yml
|
|
concurrency:
|
|
group: CI-integ-test-dependency-submission-${{ github.ref }}
|
|
cancel-in-progress: false
|
|
with:
|
|
skip-dist: false
|
|
secrets: inherit
|
|
|
|
# Aggregate gate: a single check that succeeds only when all integ-test jobs succeed.
|
|
# Require this one check in branch protection instead of every fanned-out matrix job.
|
|
integ-test-success:
|
|
if: ${{ always() }}
|
|
needs:
|
|
- build-distribution
|
|
- caching-integ-tests
|
|
- other-integ-tests
|
|
- dependency-submission-integ-tests
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Fail if any integ-test job failed or was cancelled
|
|
if: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}
|
|
run: |
|
|
echo "One or more integ-test jobs did not succeed:"
|
|
echo " build-distribution: ${{ needs.build-distribution.result }}"
|
|
echo " caching-integ-tests: ${{ needs.caching-integ-tests.result }}"
|
|
echo " other-integ-tests: ${{ needs.other-integ-tests.result }}"
|
|
echo " dependency-submission-integ-tests: ${{ needs.dependency-submission-integ-tests.result }}"
|
|
exit 1
|