Skip to content

Use explicit Rust setup inputs #150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
timkpaine wants to merge 1 commit into
base: main
Choose a base branch
from
+23 −7
Open

Use explicit Rust setup inputs #150

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion rust/.github/workflows/build.yaml.jinja
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,12 @@ jobs:
with:
version: {% raw %}${{ matrix.python-version }}{% endraw %}

- uses: actions-ext/rust/setup@c9cdbe5cc8a2485bf2eb22644302559c741ca763
- uses: actions-ext/rust/setup@7d919b14e08dc1155a267dffbcc58260bdfdd63b
with:
targets: {% raw %}${{ runner.os == 'macOS' && 'aarch64-apple-darwin' || '' }}{% endraw %}
tools: |
cargo-nextest@0.9.140
cargo-llvm-cov@0.8.7

- name: Install dependencies
run: make develop
Expand Down
6 changes: 4 additions & 2 deletions rust/rust/Makefile.jinja
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,12 @@
CARGO_NEXTEST_VERSION ?= 0.9.140
CARGO_LLVM_COV_VERSION ?= 0.8.7

.PHONY: requirements develop build
requirements: ## install required dev dependencies
rustup component add rustfmt
rustup component add clippy
cargo install --locked --force cargo-nextest
cargo install --force cargo-llvm-cov
command -v cargo-nextest >/dev/null 2>&1 || cargo install --locked cargo-nextest --version =$(CARGO_NEXTEST_VERSION)
command -v cargo-llvm-cov >/dev/null 2>&1 || cargo install --locked cargo-llvm-cov --version =$(CARGO_LLVM_COV_VERSION)

develop: requirements ## install required dev dependencies

Expand Down
8 changes: 7 additions & 1 deletion rustjswasm/.github/workflows/build.yaml.jinja
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,13 @@ jobs:
with:
version: {% raw %}${{ matrix.python-version }}{% endraw %}

- uses: actions-ext/rust/setup@c9cdbe5cc8a2485bf2eb22644302559c741ca763
- uses: actions-ext/rust/setup@7d919b14e08dc1155a267dffbcc58260bdfdd63b
with:
targets: {% raw %}${{ runner.os == 'macOS' && 'aarch64-apple-darwin,wasm32-unknown-unknown' || 'wasm32-unknown-unknown' }}{% endraw %}
tools: |
cargo-nextest@0.9.140
cargo-llvm-cov@0.8.7
wasm-bindgen-cli@0.2.126

- uses: actions-ext/node/setup@3478453a763ed3c8c6eba05a092ab81a265c5492
with:
Expand Down
9 changes: 6 additions & 3 deletions rustjswasm/rust/Makefile.jinja
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
CARGO_NEXTEST_VERSION ?= 0.9.140
CARGO_LLVM_COV_VERSION ?= 0.8.7
WASM_BINDGEN_CLI_VERSION ?= 0.2.126

.PHONY: requirements develop build
requirements: ## install required dev dependencies
rustup component add rustfmt
rustup component add clippy
cargo install --force --locked cargo-nextest
cargo install --force --locked cargo-llvm-cov
cargo install --force --locked wasm-bindgen-cli --version 0.2.126
command -v cargo-nextest >/dev/null 2>&1 || cargo install --locked cargo-nextest --version =$(CARGO_NEXTEST_VERSION)
command -v cargo-llvm-cov >/dev/null 2>&1 || cargo install --locked cargo-llvm-cov --version =$(CARGO_LLVM_COV_VERSION)
command -v wasm-bindgen >/dev/null 2>&1 || cargo install --locked wasm-bindgen-cli --version =$(WASM_BINDGEN_CLI_VERSION)
rustup target add wasm32-unknown-unknown

develop: requirements ## install required dev dependencies
Expand Down
Loading