killzoner commented on code in PR #1327:
URL: 
https://github.com/apache/datafusion-ballista/pull/1327#discussion_r2428388483


##########
.github/workflows/rust.yml:
##########
@@ -79,15 +81,53 @@ jobs:
           CARGO_HOME: "/github/home/.cargo"
           CARGO_TARGET_DIR: "/github/home/target"
 
+  # Run `cargo test doc` (test documentation examples)
+  linux-test-doc:
+    name: cargo test doc (amd64)
+    needs: linux-build-lib
+    runs-on: ubuntu-latest
+    container:
+      image: amd64/rust
+    steps:
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8  # 
v5.0.0

Review Comment:
   Sorry that's a miss. Basically `datafusion` uses this hash (with the 
comment) everywhere in the CI, so I started replicating and figured out it 
might be cleaner to just point to v5
   
   I will have a look at the failing doc test, I might just remove it for now 
as I copied it from upstream `datafusion` and it's not serving any specific 
purpose at least for this PR



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to