kszucs opened a new pull request, #2587:
URL: https://github.com/apache/iceberg-rust/pull/2587

   ## Which issue does this PR close?
   
   - N/A (CI-only cleanup)
   
   ## What changes are included in this PR?
   
   `ci_hf_cdc.yml` mostly duplicated the main CI jobs; its only real purpose 
was injecting the HuggingFace secrets. The tests it ran are already collected 
by the existing jobs and already self-skip when secrets are absent (the Rust 
`require_env!` macro and the Python `@requires_hf` skipif). So:
   
   - **`ci.yml`** — expose `HF_TOKEN` / `HF_BUCKET` / `HF_DATASET` to the 
existing `tests` job.
   - **`bindings_python_ci.yml`** — expose `HF_TOKEN` / `HF_DATASET` to the 
existing `test` job.
   - **`ci_hf_cdc.yml`** — deleted.
   
   This matches how the S3/GCS/Glue/HMS integration tests already run inside 
the main job. Forks skip cleanly via the in-test guards.
   
   ## Are these changes tested?
   
   Covered by the existing HF/CDC integration tests, now driven by the main CI 
jobs. No test logic changed — only where the secrets are injected. Rust HF 
tests now run under parallel `nextest` (previously pinned to 
`--test-threads=1`); if collisions surface, a nextest test-group can serialize 
`file_io_hf_test`.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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