-
Notifications
You must be signed in to change notification settings - Fork 82
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: Add CI status and change build-check to ci (#833)
* fix: Add CI status and change build-check to ci Align zenoh-pico with the rest of the zenoh ecosystem in terms of CI, - rename build-check.yml to ci.yml - Include CI status check to ci.yml * chore: fix test name * fix: experiment with reusable workflow * fix: make integration and build-static reusable * fix: make multicast tests reusable * fix: typo in job name
- Loading branch information
1 parent
e40c236
commit 197ab2c
Showing
5 changed files
with
38 additions
and
22 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,13 +11,15 @@ | |
# Contributors: | ||
# ZettaScale Zenoh Team, <[email protected]> | ||
# | ||
name: build-check | ||
name: CI | ||
|
||
on: | ||
push: | ||
branches: ["**"] | ||
pull_request: | ||
branches: ["**"] | ||
schedule: | ||
- cron: "0 6 * * 1-5" | ||
|
||
jobs: | ||
run_tests: | ||
|
@@ -282,7 +284,7 @@ jobs: | |
if: always() | ||
run: kill ${{ steps.run-zenoh.outputs.zenohd-pid }} | ||
|
||
no_routeur: | ||
no_router: | ||
name: Test examples without router | ||
runs-on: ubuntu-latest | ||
steps: | ||
|
@@ -304,3 +306,33 @@ jobs: | |
with: | ||
config: '.markdownlint.yaml' | ||
globs: '**/README.md' | ||
|
||
build_shared: | ||
name: Build shared libs | ||
uses: ./.github/workflows/build-shared.yaml | ||
|
||
build_static: | ||
name: Build static libs | ||
uses: ./.github/workflows/build-static.yaml | ||
|
||
integration: | ||
name: Run integration tests | ||
uses: ./.github/workflows/integration.yaml | ||
|
||
multicast: | ||
name: Run multicast tests | ||
uses: ./.github/workflows/multicast.yaml | ||
|
||
# NOTE: In GitHub repository settings, the "Require status checks to pass | ||
# before merging" branch protection rule ensures that commits are only merged | ||
# from branches where specific status checks have passed. These checks are | ||
# specified manually as a list of workflow job names. Thus we use this extra | ||
# job to signal whether all CI checks have passed. | ||
ci: | ||
name: CI status checks | ||
runs-on: ubuntu-latest | ||
needs: [run_tests, check_format, c99_build, raweth_build, zenoh_build, modular_build, unstable_build, st_build, fragment_test, attachment_test, memory_leak_test, no_router, markdown_lint, build_shared, build_static, integration, multicast] | ||
if: always() | ||
steps: | ||
- name: Check whether all jobs pass | ||
run: echo '${{ toJson(needs) }}' | jq -e 'all(.result == "success")' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters