Skip to content

Commit

Permalink
Install toolchain with rustup directly
Browse files Browse the repository at this point in the history
Signed-off-by: Joe Richey <joerichey@google.com>
  • Loading branch information
josephlr committed Jan 4, 2021
1 parent 7b8f2ba commit fdac87f
Showing 1 changed file with 11 additions and 15 deletions.
26 changes: 11 additions & 15 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,11 +70,10 @@ jobs:
]
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
with:
profile: minimal
target: ${{ matrix.target }}
toolchain: stable
- name: Install toolchain/target
run: |
rustup update stable --no-self-update --profile=minimal --target=${{ matrix.target }}
rustup default stable
- name: Install multilib
# update is needed to fix the 404 error on install, see:
# https://github.com/actions/virtual-environments/issues/675
Expand All @@ -97,11 +96,10 @@ jobs:
]
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
with:
profile: minimal
target: ${{ matrix.target }}
toolchain: stable
- name: Install toolchain/target
run: |
rustup update stable --no-self-update --profile=minimal --target=${{ matrix.target }}
rustup default stable
- name: Build Tests
run: cargo test --no-run --target=${{ matrix.target }} --features=std

Expand All @@ -118,11 +116,9 @@ jobs:
steps:
- uses: actions/checkout@v2
- name: Install toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: ${{ matrix.toolchain }}
override: true
run: |
rustup update ${{ matrix.toolchain }} --no-self-update --profile=minimal
rustup default ${{ matrix.toolchain }}
- run: cargo test --features=std

# TODO: Add emscripten when it's working with Cross
Expand Down

0 comments on commit fdac87f

Please sign in to comment.