From 13cfcae5a443978f2b6364fe8fa9d5281882456f Mon Sep 17 00:00:00 2001 From: prql-bot <107324867+prql-bot@users.noreply.github.com> Date: Tue, 2 Jun 2026 08:07:17 +0000 Subject: [PATCH] ci: remove dead --no-deps ternary in cargo doc step The step's `if:` already gates on `inputs.nightly == 'true'`, so the `${{ inputs.nightly != 'true' && '--no-deps' || '' }}` ternary always evaluates to the empty string. Drop the dead expression and the now-stale comment. Behavior is unchanged. --- .github/workflows/test-rust.yaml | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/.github/workflows/test-rust.yaml b/.github/workflows/test-rust.yaml index 1f2659fbe814..19e3ea6a7aa8 100644 --- a/.github/workflows/test-rust.yaml +++ b/.github/workflows/test-rust.yaml @@ -183,12 +183,9 @@ jobs: uses: clechasseur/rs-cargo@v4 with: command: doc - # Only run with deps on nightly, since it's much slower, and so farâ„¢ - # we haven't seen any errors. args: --target=${{ inputs.target }} --no-default-features --features=${{ - inputs.features }} ${{ inputs.nightly != 'true' && '--no-deps' || '' - }} + inputs.features }} - name: Build extra targets for cache # When building the cache, we also run with `--all-targets` so that # prqlc builds can use the same cache.