alamb commented on code in PR #15075: URL: https://github.com/apache/datafusion/pull/15075#discussion_r1990003945
########## .github/workflows/rust.yml: ########## @@ -180,6 +180,36 @@ jobs: run: rustup toolchain install stable - name: Install Protobuf Compiler run: sudo apt-get install -y protobuf-compiler + - name: Run tests (excluding doctests and datafusion-cli) + env: + RUST_BACKTRACE: 1 + run: | + cargo test \ + --profile ci \ + --exclude datafusion-examples \ + --exclude ffi_example_table_provider \ + --exclude datafusion-benchmarks \ + --exclude datafusion-cli \ + --workspace \ + --lib \ + --tests \ + --bins \ + --features avro,json,backtrace,integration-tests + - name: Verify Working Directory Clean + run: git diff --exit-code + + # datafusion-cli tests + linux-test-datafusion-cli: Review Comment: ~It was not -- I will remove it. Good point~ Update: I actually think it is good to have the mac tests still run (even if not integration tests) to provide basic coverage -- 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. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org