Skip to content

Commit

Permalink
Split out ARM64 test-fast from matrix, make nonrequired
Browse files Browse the repository at this point in the history
As suggested in:
#1790 (comment)

It likely won't have to be kept this way. But making it nonrequired
for now makes it so that investigating what triggers the SIGSEGV
(and SIGBUS) errors -- as well as other errors that were found
while investigating that (d9e7fdb, e71b0cf, 5a71963) -- doesn't
have to be rushed.
  • Loading branch information
EliahKagan committed Jan 22, 2025
1 parent 97f85f7 commit 50da7cb
Showing 1 changed file with 20 additions and 1 deletion.
21 changes: 20 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@ jobs:
- windows-latest
- macos-latest
- ubuntu-latest
- ubuntu-24.04-arm

runs-on: ${{ matrix.os }}

Expand All @@ -161,6 +160,25 @@ jobs:
- name: Check that tracked archives are up to date
run: git diff --exit-code # If this fails, the fix is usually to commit a regenerated archive.

test-fast-arm:
runs-on: ubuntu-24.04-arm

steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- uses: Swatinem/rust-cache@v2
- uses: taiki-e/install-action@v2
with:
tool: nextest
- name: Test (nextest)
env:
GIX_TEST_CREATE_ARCHIVES_EVEN_ON_CI: '1'
run: cargo nextest run --workspace --no-fail-fast
- name: Doctest
run: cargo test --workspace --doc --no-fail-fast
- name: Check that tracked archives are up to date
run: git diff --exit-code # If this fails, the fix is usually to commit a regenerated archive.

test-fixtures-windows:
runs-on: windows-latest

Expand Down Expand Up @@ -387,6 +405,7 @@ jobs:
env:
# List all jobs that are intended NOT to block PR auto-merge here.
EXPECTED_NONBLOCKING_JOBS: |-
test-fast-arm
test-fixtures-windows
cargo-deny-advisories
wasm
Expand Down

0 comments on commit 50da7cb

Please sign in to comment.