ci: clearer and cleaner steps
This commit is contained in:
parent
87b4e3b7fe
commit
6af331bd93
1 changed files with 6 additions and 4 deletions
10
.github/workflows/ci.yml
vendored
10
.github/workflows/ci.yml
vendored
|
|
@ -31,6 +31,7 @@ env:
|
||||||
jobs:
|
jobs:
|
||||||
pr-comment-flags:
|
pr-comment-flags:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
needs: [lint-cmake, lint-clang]
|
||||||
if: ${{ github.event_name == 'pull_request' }}
|
if: ${{ github.event_name == 'pull_request' }}
|
||||||
|
|
||||||
outputs:
|
outputs:
|
||||||
|
|
@ -57,7 +58,7 @@ jobs:
|
||||||
# Quality gate to allow PR merge, used in the branch protection rules.
|
# Quality gate to allow PR merge, used in the branch protection rules.
|
||||||
ci-passed:
|
ci-passed:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
needs: [lint-cmake, lint-clang, windows, macos, linux, unix]
|
needs: [test-results, unix]
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- run: echo "✅ CI passed" > $GITHUB_STEP_SUMMARY
|
- run: echo "✅ CI passed" > $GITHUB_STEP_SUMMARY
|
||||||
|
|
@ -106,7 +107,7 @@ jobs:
|
||||||
uses: ./.github/workflows/valgrind-analysis.yml
|
uses: ./.github/workflows/valgrind-analysis.yml
|
||||||
|
|
||||||
analyse-sonarcloud:
|
analyse-sonarcloud:
|
||||||
needs: [lint-cmake, lint-clang, pr-comment-flags]
|
needs: pr-comment-flags
|
||||||
if: ${{ needs.pr-comment-flags.outputs.no-sonar != 'true' }}
|
if: ${{ needs.pr-comment-flags.outputs.no-sonar != 'true' }}
|
||||||
uses: ./.github/workflows/sonarcloud-analysis.yml
|
uses: ./.github/workflows/sonarcloud-analysis.yml
|
||||||
secrets:
|
secrets:
|
||||||
|
|
@ -282,6 +283,7 @@ jobs:
|
||||||
path: ${{ env.PACKAGE_PATH }}
|
path: ${{ env.PACKAGE_PATH }}
|
||||||
|
|
||||||
linux-matrix:
|
linux-matrix:
|
||||||
|
needs: [lint-cmake, lint-clang]
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|
||||||
outputs:
|
outputs:
|
||||||
|
|
@ -300,7 +302,7 @@ jobs:
|
||||||
jq-filter: .distro |= map(select(.["runs-on"] | contains("arm64") | not))
|
jq-filter: .distro |= map(select(.["runs-on"] | contains("arm64") | not))
|
||||||
|
|
||||||
linux:
|
linux:
|
||||||
needs: [linux-matrix, lint-cmake, lint-clang]
|
needs: linux-matrix
|
||||||
name: linux-${{ matrix.distro.name }}
|
name: linux-${{ matrix.distro.name }}
|
||||||
runs-on: ${{ matrix.distro.runs-on }}
|
runs-on: ${{ matrix.distro.runs-on }}
|
||||||
container: ${{ matrix.distro.container }}
|
container: ${{ matrix.distro.container }}
|
||||||
|
|
@ -401,7 +403,7 @@ jobs:
|
||||||
|
|
||||||
release_assets:
|
release_assets:
|
||||||
name: Release
|
name: Release
|
||||||
needs: [windows, linux, macos]
|
needs: ci-passed
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- name: Download Files
|
- name: Download Files
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue