You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/01/22 13:16:59 UTC

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

nevi-me commented on a change in pull request #9262:
URL: https://github.com/apache/arrow/pull/9262#discussion_r562623291



##########
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:
       A solution could be to exclude arrow from the initial `cargo test`, then test it after `cd arrow`, with the prettyprint (and other future) feature.
   
   Or has this been resolved?




----------------------------------------------------------------
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