[GitHub] [arrow] jorgecarleitao commented on a change in pull request #9262: ARROW-11317: [Rust] Include the prettyprint feature in CI Coverage

[GitBox <gi...@apache.org>]

jorgecarleitao commented on a change in pull request #9262:
URL: https://github.com/apache/arrow/pull/9262#discussion_r562857366



##########
File path: .github/workflows/rust.yml
##########
@@ -109,14 +109,14 @@ jobs:
           export CARGO_HOME="/github/home/.cargo"
           export CARGO_TARGET_DIR="/github/home/target"
           cd rust
-          cargo test
+          # run tests on all workspace members
+          cargo test --all --features=prettyprint
           # test datafusion examples
           cd datafusion
           cargo run --example csv_sql
           cargo run --example parquet_sql
           cd ..
           cd arrow
-          cargo test

Review comment:
       To be fair, running the arrow tests is a second or so, so the harm is pretty small. Re-compiling DataFusion's dependencies would have been a different beast ^_^




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org