Skip to content

Conversation

@Lenvanderhof
Copy link
Contributor

Summary

  • Align CI docs job with [package.metadata.docs.rs] configuration
  • Use --features compression instead of --all-features to avoid features requiring system deps not available on docs.rs (python, local-embeddings)

Changes

  • Updated .github/workflows/ci.yml docs job to use --features compression

Test plan

  • CI docs job passes
  • docs.rs build succeeds with matching feature flags

🤖 Generated with Claude Code

…onfig

The docs.rs build uses `features = ["compression"]` in Cargo.toml metadata.
Using --all-features fails because python/local-embeddings require
system dependencies not available on docs.rs.

This aligns the CI docs job with the docs.rs configuration.
Copilot AI review requested due to automatic review settings January 10, 2026 22:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aligns the CI documentation build job with the docs.rs metadata configuration to ensure the CI environment accurately reflects what will be built on docs.rs. The change switches from using --all-features to --features compression to avoid features that require system dependencies not available on docs.rs.

Changes:

  • Updated CI docs job to use --features compression instead of --all-features
  • Added explanatory comments about why python and local-embeddings features are excluded

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Lenvanderhof Lenvanderhof merged commit 503f335 into main Jan 10, 2026
10 of 26 checks passed
@Lenvanderhof Lenvanderhof deleted the fix/docs-rs-build-v2 branch January 10, 2026 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant