From 9ab6eb497c428906b6a13a66c7e08f21dbf85677 Mon Sep 17 00:00:00 2001 From: Angus Hollands Date: Thu, 29 Sep 2022 07:06:25 +0100 Subject: [PATCH 1/5] ci: run build-test on main --- .github/workflows/build-test.yml | 4 ++++ .github/workflows/tests.yml | 3 +++ 2 files changed, 7 insertions(+) diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index c23362efd4..44016921c5 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -1,6 +1,10 @@ name: Build Test on: + push: + branches: + - main + pull_request: paths-ignore: - README.md diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 8903a3c6da..57a50b91e6 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -3,6 +3,9 @@ name: Tests on: pull_request: workflow_dispatch: + push: + branches: + - main concurrency: From fbb6ed5bb8f9be912fc4a22ccb9b26ffeed0f860 Mon Sep 17 00:00:00 2001 From: Angus Hollands Date: Thu, 29 Sep 2022 07:22:33 +0100 Subject: [PATCH 2/5] chore: remove cron schedule for build-test Now we test each commit to main, this isn't required --- .github/workflows/build-test.yml | 3 --- 1 file changed, 3 deletions(-) diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index 44016921c5..36e1e6d2c3 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -21,9 +21,6 @@ on: - src/cuda-kernels - cuda-kernels - schedule: - - cron: 0 12 1 * * - workflow_dispatch: concurrency: From 9504cdf67fa15ee00438f4cc319190f40e3ac5c8 Mon Sep 17 00:00:00 2001 From: Angus Hollands Date: Thu, 29 Sep 2022 14:24:29 +0100 Subject: [PATCH 3/5] ci: add Codecov workflow --- .github/workflows/coverage.yml | 72 ++++++++++++++++++++++++++++++++++ 1 file changed, 72 insertions(+) create mode 100644 .github/workflows/coverage.yml diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml new file mode 100644 index 0000000000..1560580daa --- /dev/null +++ b/.github/workflows/coverage.yml @@ -0,0 +1,72 @@ +name: Codecov + +on: + push: + branches: + - main + paths-ignore: + - README*.md + - VERSION_INFO + - .ci/* + - .readthedocs.yml + - include/* + - src/* + - docs-src/* + - docs-img/* + - docs-jupyter/* + - docs-doxygen/* + - docs-sphinx/* + - studies/* + + workflow_dispatch: + +concurrency: + group: 'coverage-${{ github.head_ref || github.run_id }}' + cancel-in-progress: true + +jobs: + coverage: + runs-on: ubuntu-20.04 + + env: + PIP_ONLY_BINARY: cmake + PYTHON_VERSION: "3.10" + + steps: + - uses: actions/checkout@v3 + with: + submodules: true + + - name: Setup ccache + uses: hendrikmuhs/ccache-action@v1.2 + with: + # Maybe share these across workflows? + key: >- + ${{ github.job }} + + - name: Use ccache + run: | + echo "/usr/lib/ccache" >> $GITHUB_PATH + echo "/usr/local/opt/ccache/libexec" >> $GITHUB_PATH + + - name: 'Python ${{ env.PYTHON_VERSION }}' + uses: actions/setup-python@v4 + with: + python-version: '${{ env.PYTHON_VERSION }}' + + - name: Install NumPy + run: python -m pip install numpy + + - name: Build + run: 'python -m pip install -v .[test,dev]' + + - name: Print versions + run: python -m pip list + + - name: Test + run: >- + python -m pytest -vv -rs tests --cov=awkward --cov-report=term + --cov-report=xml + + - name: Upload Codecov results + uses: codecov/codecov-action@v3 From f6e433294789dab6ea7611e6d4e6e4f57cfdc112 Mon Sep 17 00:00:00 2001 From: Angus Hollands Date: Thu, 29 Sep 2022 14:25:19 +0100 Subject: [PATCH 4/5] Revert "chore: remove cron schedule for build-test" This reverts commit fbb6ed5bb8f9be912fc4a22ccb9b26ffeed0f860. --- .github/workflows/build-test.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index 36e1e6d2c3..44016921c5 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -21,6 +21,9 @@ on: - src/cuda-kernels - cuda-kernels + schedule: + - cron: 0 12 1 * * + workflow_dispatch: concurrency: From 85b2e227ccbb366edf86a36b67fbe69d4c7fd76e Mon Sep 17 00:00:00 2001 From: Angus Hollands Date: Thu, 29 Sep 2022 14:26:52 +0100 Subject: [PATCH 5/5] Revert "ci: run build-test on main" This reverts commit 9ab6eb497c428906b6a13a66c7e08f21dbf85677. --- .github/workflows/build-test.yml | 4 ---- .github/workflows/tests.yml | 3 --- 2 files changed, 7 deletions(-) diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml index 44016921c5..c23362efd4 100644 --- a/.github/workflows/build-test.yml +++ b/.github/workflows/build-test.yml @@ -1,10 +1,6 @@ name: Build Test on: - push: - branches: - - main - pull_request: paths-ignore: - README.md diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 57a50b91e6..8903a3c6da 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -3,9 +3,6 @@ name: Tests on: pull_request: workflow_dispatch: - push: - branches: - - main concurrency: