From 4c5e30a9f83b4d856e58d23a4e89d662abf476ad Mon Sep 17 00:00:00 2001 From: Alice Cecile Date: Sat, 2 Jul 2022 00:55:52 +0000 Subject: [PATCH] Re-enable check-unused-dependencies in CI (#5172) # Objective Fixes #5155. This *should* work now that the semver breaking dependency of the CI crate got yanked, but we'll see what CI has to say about it. --- .github/workflows/ci.yml | 48 ++++++++++++++++++++-------------------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index abb68fee29..e4e63ebaad 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -272,27 +272,27 @@ jobs: echo "if this step fails, run the following command and commit the changed file on your PR." echo " > cargo run -p build-example-pages -- update" git diff --quiet HEAD -- - # TODO: re-enable once cargo-udeps builds with changes from rust 1.62 - # check-unused-dependencies: - # runs-on: ubuntu-latest - # steps: - # - uses: actions/checkout@v3 - # - uses: actions/cache@v3 - # with: - # path: | - # ~/.cargo/bin/ - # ~/.cargo/registry/index/ - # ~/.cargo/registry/cache/ - # ~/.cargo/git/db/ - # target/ - # key: ${{ runner.os }}-cargo-check-unused-dependencies-${{ hashFiles('**/Cargo.toml') }} - # - uses: actions-rs/toolchain@v1 - # with: - # toolchain: nightly - # override: true - # - name: Installs cargo-udeps - # run: cargo install --force cargo-udeps - # - name: Install alsa and udev - # run: sudo apt-get update; sudo apt-get install --no-install-recommends libasound2-dev libudev-dev - # - name: Run cargo udeps - # run: cargo udeps + + check-unused-dependencies: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v3 + - uses: actions/cache@v3 + with: + path: | + ~/.cargo/bin/ + ~/.cargo/registry/index/ + ~/.cargo/registry/cache/ + ~/.cargo/git/db/ + target/ + key: ${{ runner.os }}-cargo-check-unused-dependencies-${{ hashFiles('**/Cargo.toml') }} + - uses: actions-rs/toolchain@v1 + with: + toolchain: nightly + override: true + - name: Installs cargo-udeps + run: cargo install --force cargo-udeps + - name: Install alsa and udev + run: sudo apt-get update; sudo apt-get install --no-install-recommends libasound2-dev libudev-dev + - name: Run cargo udeps + run: cargo udeps