From 592e7e711406ddc66a61c45bfa66ba91f1adede0 Mon Sep 17 00:00:00 2001 From: Pat Date: Thu, 19 Dec 2024 14:16:37 +0000 Subject: [PATCH] ci: add required checks and fix packages (#57) Signed-off-by: Patrick Stephens --- .github/workflows/build.yaml | 28 ++++++++++++++++++++++++++++ .github/workflows/packages.yaml | 6 +++--- CODEOWNERS | 7 +++++++ 3 files changed, 38 insertions(+), 3 deletions(-) create mode 100644 CODEOWNERS diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index dbe9d44..1ba0cbf 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -183,3 +183,31 @@ jobs: # dependencies_debian: '' cmakeflags: '-DCFL_TESTS=On -DCFL_DEV=on .' build_command: make all + + + # this job provides the single required status for PRs to be merged into main. + # instead of updating the protected branch status in github, developers can update the needs section below + # to require additional status checks to protect main. + # the job uses the alls-green action to get around the github issue that treats a "skipped" required status check + # as passed. github will skip a job if an upstream needed job fails, which would defeat the purpose of this required + # status check. + test-required-checks-complete: + # note: this step always has to run in order to check if the dependent jobs passed. by default github skips running a job + # if the needed jobs upstream failed. + if: always() + needs: + - build-windows + - build-msys2 + - build-centos + - build-debian + - build-unix-arm64 + - build-unix-amd64 + - build-analysis-tests + name: Required checks complete + runs-on: ubuntu-latest + permissions: {} + steps: + - name: Decide whether the needed jobs succeeded or failed + uses: re-actors/alls-green@release/v1 + with: + jobs: ${{ toJSON(needs) }} diff --git a/.github/workflows/packages.yaml b/.github/workflows/packages.yaml index 1ea720a..58a5425 100644 --- a/.github/workflows/packages.yaml +++ b/.github/workflows/packages.yaml @@ -19,11 +19,11 @@ jobs: format: [ rpm, deb ] steps: - uses: actions/checkout@v4 - - uses: uraimo/run-on-arch-action@v2.5.0 + - uses: uraimo/run-on-arch-action@v2.8.1 name: Build the ${{matrix.format}} packages with: arch: aarch64 - distro: ubuntu20.04 + distro: ubuntu_latest run: | apt-get update && \ apt-get install -y --no-install-recommends \ @@ -31,7 +31,7 @@ jobs: cmake \ file \ rpm \ - make + make cmake . echo ${{ matrix.format }} | awk '{print toupper($0)}' | xargs -I{} cpack -G {} diff --git a/CODEOWNERS b/CODEOWNERS new file mode 100644 index 0000000..2c2d84e --- /dev/null +++ b/CODEOWNERS @@ -0,0 +1,7 @@ +# Global Owners +# ------------- +* @edsiper @leonardo-albertovich + +# CI +# ------------------------- +/.github/ @niedbalski @patrick-stephens @celalettin1286