Skip to content

Commit 46d5b5e

Browse files
committed
Chore: exclude libs in clippy and fix workspace dir
1 parent 41af20d commit 46d5b5e

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

.github/workflows/rust.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ jobs:
1919
name: Rust ${{ matrix.rust }} on ${{ matrix.os }}
2020
runs-on: ${{ matrix.os }}
2121
env:
22+
# Temporarily exclude crates that rely on OptiX until we merge the rewritten optix because of bindings weirdness
23+
# Also exclude add until we figure out the right way to auto-add libnvvm to path.
2224
LLVM_LINK_STATIC: 1
25+
EXCLUDE_ARGS: --exclude "optix" --exclude "optix_sys" --exclude "path_tracer" --exclude "denoiser" --exclude "add"
2326
strategy:
2427
fail-fast: false
2528
matrix:
@@ -48,7 +51,7 @@ jobs:
4851
- name: Download OptiX Headers
4952
run: |
5053
chmod +x "${GITHUB_WORKSPACE}/scripts/download_ci_optix.bash"
51-
scripts/download_ci_optix.bash
54+
${GITHUB_WORKSPACE}/scripts/download_ci_optix.bash
5255
5356
- name: Install LLVM 7
5457
if: contains(matrix.os, 'ubuntu')
@@ -63,10 +66,8 @@ jobs:
6366
if: contains(matrix.os, 'ubuntu')
6467
run: cargo fmt --all -- --check
6568

66-
# Temporarily exclude crates that rely on OptiX until we merge the rewritten optix because of bindings weirdness
67-
# Also exclude add until we figure out the right way to auto-add libnvvm to path.
6869
- name: Build
69-
run: cargo build --workspace --exclude "optix" --exclude "optix_sys" --exclude "path_tracer" --exclude "denoiser" --exclude "add"
70+
run: cargo build --workspace ${EXCLUDE_ARGS}
7071

7172
# Don't currently test because many tests rely on the system having a CUDA GPU
7273
# - name: Test
@@ -76,7 +77,7 @@ jobs:
7677
if: contains(matrix.os, 'ubuntu')
7778
env:
7879
RUSTFLAGS: -Dwarnings
79-
run: cargo clippy --workspace
80+
run: cargo clippy --workspace ${EXCLUDE_ARGS}
8081

8182
- name: Check documentation
8283
env:

0 commit comments

Comments
 (0)