Skip to content

Commit

Permalink
fix(ci): install taplo through moonrepo (#2672)
Browse files Browse the repository at this point in the history
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 <gilad@starkware.com>
  • Loading branch information
giladchase and Gilad Chase authored Dec 15, 2024
1 parent fe2c664 commit 94fe7a0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
3 changes: 3 additions & 0 deletions .github/actions/install_rust/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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"
6 changes: 1 addition & 5 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 94fe7a0

Please sign in to comment.