Skip to content

Commit 4306a1c

Browse files
committed
Don't run tests on MSRV (only cargo check)
The tests break because of the `time` crate (a dev-dependency). To workaround this we only run `cargo check` on the MSRV
1 parent a1886c9 commit 4306a1c

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

.github/workflows/test.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,14 @@ jobs:
5050
with:
5151
toolchain: ${{ matrix.rust }}
5252
override: true
53-
# NOTE: We only run `cargo test`. No need for a seperate `cargo check`
53+
- name: Check
54+
run: |
55+
cargo check --verbose --features "${{ matrix.features }}"
56+
# A failing `cargo check` always fails the build
57+
continue-on-error: false
58+
# NOTE: We only run `cargo test` if version > MSRV. Tests break on old MSRV due to
59+
# a dev-dependency.
5460
- name: Test
5561
run: |
5662
cargo test --verbose --features "${{ matrix.features }}"
57-
63+
if: "${{ matrix.rust == 'stable' || matrix.rust == 'nightly' }}"

0 commit comments

Comments
 (0)