From 5dfc73f353dc5337bdeadee65951eace98566b16 Mon Sep 17 00:00:00 2001 From: Leigh McCulloch <351529+leighmcculloch@users.noreply.github.com> Date: Thu, 11 Jul 2024 12:42:38 +1000 Subject: [PATCH] Remove rust-analyzer-compat ci check ### What Remove rust-analyzer-compat ci check. ### Why We added this when we seemed to be frequently breaking rust-analyzer which slowed down dev envs for devs. But it's been more than 1 year since I've seen anything break rust-analyzer. RA continues to get better, and I don't think we need to keep running this check. There have been at times problems with the check, although not frequent, it's part of our process that isn't serving the need it was created for any longer. --- .github/workflows/rust.yml | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index e46217bc92..86f2e0f966 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -11,21 +11,12 @@ jobs: complete: if: always() - needs: [rust-analyzer-compat, build-and-test, publish-dry-run] + needs: [build-and-test, publish-dry-run] runs-on: ubuntu-latest steps: - if: contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') run: exit 1 - rust-analyzer-compat: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v3 - - run: rustup update - - run: rustup +nightly component add rust-analyzer - - name: Check if rust-analyzer encounters any errors parsing project - run: rustup run nightly rust-analyzer analysis-stats . 2>&1 | (! grep ERROR) - build-and-test: strategy: matrix: