From a6023a52349f35f533fcb7efee42119ee9ccca31 Mon Sep 17 00:00:00 2001 From: Niklas Mohrin Date: Sun, 21 Aug 2022 22:13:32 +0200 Subject: [PATCH] Small fixes in `ci.yml` - Add `profile: minimal` - Install `rustfmt` when installing the toolchain, not afterwards - Use stable toolchain for `rustfmt` - Replace `override` with `default`. I think this better reflects the "use what we just installed" intent. --- .github/workflows/ci.yml | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index bfaa6db..8f53a47 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -9,7 +9,6 @@ on: name: CI jobs: - test: name: run tests strategy: @@ -22,7 +21,8 @@ jobs: - uses: actions-rs/toolchain@v1 with: toolchain: ${{ matrix.rust }} - override: true + profile: minimal + default: true - name: Build with default features uses: actions-rs/cargo@v1 with: @@ -45,8 +45,9 @@ jobs: - uses: actions-rs/toolchain@v1 with: toolchain: stable + profile: minimal + default: true components: clippy - override: true - uses: actions-rs/clippy-check@v1 with: token: ${{ secrets.GITHUB_TOKEN }} @@ -59,9 +60,10 @@ jobs: - uses: actions/checkout@v2 - uses: actions-rs/toolchain@v1 with: - toolchain: '1.60' - override: true - - run: rustup component add rustfmt + toolchain: stable + profile: minimal + default: true + components: rustfmt - uses: actions-rs/cargo@v1 with: command: fmt @@ -79,7 +81,8 @@ jobs: - uses: actions-rs/toolchain@v1 with: toolchain: stable - override: true + profile: minimal + default: true - name: Build uses: actions-rs/cargo@v1 with: