Skip to content

Commit

Permalink
Use Cargo.lock.msrv in CI tests
Browse files Browse the repository at this point in the history
This will prevent our CI breaking due to updates to our dependencies.

Because of the way the msrv.yml file is constructed, it was not
straightfoward to use the committed lockfile only for the MSRV tests.

It would be better to use a different lockfile, with latest versions
of our dependencies, for non-MSRV tests.

Signed-off-by: Ian Jackson <[email protected]>
  • Loading branch information
ijackson committed Nov 6, 2023
1 parent 3ed6f00 commit e959be8
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions .github/workflows/msrv.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,9 @@ jobs:
minimal: true
override: true

- name: Install Cargo.lock.msrv
run: cp Cargo.lock.msrv Cargo.lock

- name: Run cargo check
if: matrix.rust != 'nightly'
uses: actions-rs/cargo@v1
Expand Down Expand Up @@ -60,6 +63,9 @@ jobs:
minimal: true
override: true

- name: Install Cargo.lock.msrv
run: cp Cargo.lock.msrv Cargo.lock

- name: Run cargo test
if: matrix.rust != 'nightly' && matrix.rust != '1.59.0'
uses: actions-rs/cargo@v1
Expand Down Expand Up @@ -125,6 +131,9 @@ jobs:
override: true
components: clippy

- name: Install Cargo.lock.msrv
run: cp Cargo.lock.msrv Cargo.lock

- name: Run cargo clippy
uses: actions-rs/cargo@v1
with:
Expand Down Expand Up @@ -152,6 +161,9 @@ jobs:
minimal: true
override: true

- name: Install Cargo.lock.msrv
run: cp Cargo.lock.msrv Cargo.lock

- name: Run cargo check
uses: actions-rs/cargo@v1
with:
Expand Down

0 comments on commit e959be8

Please sign in to comment.