Skip to content

Commit

Permalink
Merge pull request #3 from twosigma/debian-matrix
Browse files Browse the repository at this point in the history
build the package only on the oldest debian we're supporting
  • Loading branch information
morisja authored Oct 3, 2024
2 parents 9c69284 + 14c4a09 commit 580c92d
Showing 1 changed file with 4 additions and 26 deletions.
30 changes: 4 additions & 26 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,23 +26,6 @@ jobs:
- name: Run tests
run: cargo test --verbose

coverage:
runs-on: ubuntu-latest
container:
image: xd009642/tarpaulin:develop
options: --security-opt seccomp=unconfined

steps:
- uses: actions/checkout@v4
- uses: actions-rust-lang/setup-rust-toolchain@v1
with:
toolchain: ${{ github.event.schedule && 'nightly' || 'stable' }}
- name: Generate code coverage
run: cargo tarpaulin --all-features --workspace --timeout 120 --out xml
- uses: codecov/codecov-action@v3
with:
fail_ci_if_error: true

build-debian:
strategy:
matrix:
Expand Down Expand Up @@ -78,14 +61,9 @@ jobs:
with:
token: ${{ secrets.GITHUB_TOKEN }}


build-debian-package:
strategy:
matrix:
debian-version: [11, 12]

runs-on: ubuntu-latest
container: debian:${{ matrix.debian-version }}
container: debian:11

steps:
- uses: actions/checkout@v4
Expand All @@ -107,7 +85,7 @@ jobs:
working-directory: clone
- uses: actions/upload-artifact@v3
with:
name: deb-${{ matrix.debian-version }}-package
name: deb-package
path: |
./*
!./clone/**
Expand All @@ -120,7 +98,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/download-artifact@v3
with:
name: deb-11-package
name: deb-package
- name: CI
run: ci/test.sh
env:
Expand All @@ -144,7 +122,7 @@ jobs:
apt-get -y install build-essential
- uses: actions/download-artifact@v3
with:
name: deb-${{ matrix.debian-version }}-package
name: deb-package
- name: CI
run: ci/test.sh
env:
Expand Down

0 comments on commit 580c92d

Please sign in to comment.