From 94fe7a0a6bc92577be814e69ee194792091dcee7 Mon Sep 17 00:00:00 2001 From: giladchase Date: Sun, 15 Dec 2024 15:55:18 +0200 Subject: [PATCH] fix(ci): install taplo through moonrepo (#2672) Looks like cargo-install action isn't picking up the `lld` rustflags inside main.yml. Anyhow, installing through moonrepo is probably better anyway, since it is already installing rust. Co-Authored-By: Gilad Chase --- .github/actions/install_rust/action.yml | 3 +++ .github/workflows/main.yml | 6 +----- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/.github/actions/install_rust/action.yml b/.github/actions/install_rust/action.yml index 0f35796fc8..1fe72bcc42 100644 --- a/.github/actions/install_rust/action.yml +++ b/.github/actions/install_rust/action.yml @@ -8,3 +8,6 @@ runs: with: cache-base: main(-v[0-9].*)? inherit-toolchain: true + bins: taplo-cli@0.9.0 + env: + RUSTFLAGS: "-C link-arg=-fuse-ld=lld" diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index ed65bae32e..382b7c7dc4 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -35,11 +35,7 @@ jobs: # Fetch the entire history. Required to checkout the merge target commit, so the diff can # be computed. fetch-depth: 0 - - uses: baptiste0928/cargo-install@v3 - with: - crate: taplo-cli - version: '0.9.0' - locked: true + # Setup pypy and link to the location expected by .cargo/config.toml. - uses: actions/setup-python@v5