File tree Expand file tree Collapse file tree
Expand file tree Collapse file tree Original file line number Diff line number Diff line change @@ -129,7 +129,7 @@ jobs:
129129 - uses : ./.github/actions/setup-prebuild
130130 - name : Docs
131131 run : |
132- RUSTDOCFLAGS="-D warnings" cargo doc --profile ci --no-deps
132+ RUSTDOCFLAGS="-D warnings" cargo doc --workspace -- profile ci --no-deps
133133 # nextest doesn't support doc tests, so we run it here
134134 cargo test --profile ci --doc --workspace --all-features --exclude vortex-cxx --exclude vortex-jni --exclude vortex-ffi --exclude xtask --no-fail-fast
135135
@@ -220,19 +220,19 @@ jobs:
220220 - name : Rustc check
221221 id : check
222222 continue-on-error : true
223- run : RUSTFLAGS="-D warnings" cargo check --profile ci --locked --all-features --all-targets
223+ run : RUSTFLAGS="-D warnings" cargo check --workspace -- profile ci --locked --all-features --all-targets
224224 - name : Rustc check (release)
225225 id : check-release
226226 continue-on-error : true
227- run : RUSTFLAGS="-D warnings" cargo check --locked --all-features --all-targets --release
227+ run : RUSTFLAGS="-D warnings" cargo check --workspace -- locked --all-features --all-targets --release
228228 - name : Rust Lint - Clippy All Features
229229 id : clippy-all
230230 continue-on-error : true
231- run : cargo clippy --profile ci --locked --all-features --all-targets -- -D warnings
231+ run : cargo clippy --workspace -- profile ci --locked --all-features --all-targets -- -D warnings
232232 - name : Rust Lint - Clippy Default Features
233233 id : clippy-default
234234 continue-on-error : true
235- run : cargo clippy --profile ci --locked --all-targets -- -D warnings
235+ run : cargo clippy --workspace -- profile ci --locked --all-targets -- -D warnings
236236 - name : Check lint results
237237 if : always()
238238 uses : actions/github-script@3a2844b7e9c422d3c10d287c895573f7108da1b3 # v9
Original file line number Diff line number Diff line change @@ -45,6 +45,11 @@ broad refactor:
4545cargo build --workspace
4646```
4747
48+ cargo build, cargo clippy, cargo nextest run without -p or --workspace don't
49+ cover crates that are not in "default-members" in Cargo.toml. If you change
50+ crates that are not in "default-members", build and test them using "-p" or
51+ "--workspace".
52+
4853## Testing
4954
5055Run tests for the crate or binding you touched before broader checks:
Original file line number Diff line number Diff line change @@ -70,6 +70,52 @@ members = [
7070 " vortex-geo" ,
7171]
7272exclude = [" java/testfiles" , " wasm-test" ]
73+
74+ # Benchmark harnesses and fuzzing account for 40% of a cold workspace build
75+ # and they aren't needed for ordinary builds.
76+ default-members = [
77+ " vortex-array-macros" ,
78+ " vortex-error" ,
79+ " vortex-buffer" ,
80+ " vortex-mask" ,
81+ " vortex-utils" ,
82+ " vortex-session" ,
83+ " vortex-flatbuffers" ,
84+ " vortex-metrics" ,
85+ " vortex-io" ,
86+ " vortex-proto" ,
87+ " vortex-array" ,
88+ " vortex-tensor" ,
89+ " vortex-json" ,
90+ " vortex-compressor" ,
91+ " vortex-btrblocks" ,
92+ " vortex-layout" ,
93+ " vortex-scan" ,
94+ " vortex-file" ,
95+ " vortex-ipc" ,
96+ " vortex" ,
97+ " vortex-datafusion" ,
98+ " vortex-duckdb" ,
99+ " vortex-cuda" ,
100+ " vortex-cuda/cub" ,
101+ " vortex-cuda/ffi" ,
102+ " vortex-cuda/gpu-scan-cli" ,
103+ " vortex-cuda/macros" ,
104+ " vortex-cuda/nvcomp" ,
105+ " vortex-cxx" ,
106+ " vortex-ffi" ,
107+ " vortex-jni" ,
108+ " vortex-python" ,
109+ " vortex-tui" ,
110+ " vortex-web/crate" ,
111+ " vortex-sqllogictest" ,
112+ " vortex-test/compat-gen" ,
113+ " vortex-test/e2e-cuda" ,
114+ " xtask" ,
115+ " encodings/*" ,
116+ " encodings/experimental/*" ,
117+ " vortex-geo" ,
118+ ]
73119resolver = " 2"
74120
75121[workspace .package ]
You can’t perform that action at this time.
0 commit comments