Skip to content

Commit 77c2779

Browse files
authored
Add a MSRV CI job (dora-rs#1094)
* Add a MSVR CI job Check that we don't use features that are only available on newer Rust versions. * Add `rust-version` field to node hub packages * Exclude `dora-rav1e` in `msrv` test job * Add a name for the msrv job * Exclude `dora-dav1d` in `msrv` test job
1 parent b7bfe30 commit 77c2779

File tree

3 files changed

+11
-0
lines changed

3 files changed

+11
-0
lines changed

.github/workflows/ci.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -546,3 +546,12 @@ jobs:
546546
use-cross: true
547547
command: check
548548
args: --target ${{ matrix.platform.target }} --all --exclude dora-dav1d --exclude dora-rav1e --exclude dora-node-api-python --exclude dora-operator-api-python --exclude dora-ros2-bridge-python
549+
550+
# copied from https://github.com/rust-lang/cargo/blob/6833aa715d724437dc1247d0166afe314ab6854e/.github/workflows/main.yml#L291
551+
msrv:
552+
name: "Check for specified `rust-version`"
553+
runs-on: ubuntu-latest
554+
steps:
555+
- uses: actions/checkout@v4
556+
- uses: taiki-e/install-action@cargo-hack
557+
- run: cargo hack check --all-targets --rust-version --workspace --ignore-private --locked --exclude dora-rav1e --exclude dora-dav1d

node-hub/dora-dav1d/Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ name = "dora-dav1d"
33
edition.workspace = true
44
license = "BSD-2-Clause"
55
version.workspace = true
6+
rust-version.workspace = true
67

78
# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
89

node-hub/dora-mistral-rs/Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
name = "dora-mistral-rs"
33
version = "0.1.0"
44
edition = "2024"
5+
rust-version.workspace = true
56

67
# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
78

0 commit comments

Comments
 (0)