Skip to content

[branch-50] Backport: Fix docs.rs build: Replace auto_doc_cfg with doc_cfg - #17890

Merged
alamb merged 2 commits into
apache:branch-50from
AdamGS:adamg/backport-doc-cfg-fix
Oct 3, 2025
Merged

[branch-50] Backport: Fix docs.rs build: Replace auto_doc_cfg with doc_cfg#17890
alamb merged 2 commits into
apache:branch-50from
AdamGS:adamg/backport-doc-cfg-fix

Conversation

@AdamGS

@AdamGSAdamGS commented Oct 2, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes#17889

Fixes an issue in building docs, which are broken in the 50.1.0 release.

Rationale for this change

Fixes docs build.

What changes are included in this PR?

Use doc_cfg instead of auto_doc_cfg.

Are these changes tested?

Yes, see original PR

Are there any user-facing changes?

No

@github-actionsgithub-actionsBot added sql SQL Planner logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates optimizer Optimizer rules core Core DataFusion crate sqllogictest SQL Logic Tests (.slt) substrait Changes to the substrait crate catalog Related to the catalog crate common Related to common crate execution Related to the execution crate proto Related to proto crate functions Changes to functions implementation datasource Changes to the datasource crate ffi Changes to the ffi crate physical-plan Changes to the physical-plan crate spark labels Oct 2, 2025
@AdamGSAdamGS mentioned this pull request Oct 2, 2025
Closed
22 tasks
alamb
alamb approved these changes Oct 2, 2025
View reviewed changes

@alambalamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @AdamGS 🙏

FYI @mbrobbel

@alamb

alamb commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Updated to get changes from #17892

@alamb
alamb merged commit 842d0e4 into apache:branch-50Oct 3, 2025
28 checks passed
@alamb

alamb commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

🚀 -- than you @AdamGS

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alambalambalamb approved these changes

Assignees

No one assigned

Labels

catalogRelated to the catalog cratecommonRelated to common cratecoreCore DataFusion cratedatasourceChanges to the datasource crateexecutionRelated to the execution crateffiChanges to the ffi cratefunctionsChanges to functions implementationlogical-exprLogical plan and expressionsoptimizerOptimizer rulesphysical-exprChanges to the physical-expr cratesphysical-planChanges to the physical-plan crateprotoRelated to proto cratesparksqlSQL PlannersqllogictestSQL Logic Tests (.slt)substraitChanges to the substrait crate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AdamGS@alamb@mbrobbel