Skip to content

Show members for all features in docs#87

Merged
fpagliughi merged 1 commit into
socketcan-rs:masterfrom
wojciech-graj:master
May 27, 2026
Merged

Show members for all features in docs#87
fpagliughi merged 1 commit into
socketcan-rs:masterfrom
wojciech-graj:master

Conversation

@wojciech-graj
Copy link
Copy Markdown
Contributor

@wojciech-graj wojciech-graj commented Oct 3, 2025

The docs.rs documentation is currently not very useful, as it doesn't show anything that is conditionally compiled by enabling non-default features. With this PR all members are always shown, with an optional note next to them indicating if they are feature-gated.

Test this change by generating the docs locally with the following cargo command

RUSTDOCFLAGS="--cfg docsrs" cargo +nightly doc --all-features

Some of the dependencies' documentation is borked because of rust-lang/rust#43781 which interferes with local documentation builds. I don't know docs.rs handles this; it's possible these dependencies will have to be bumped.

Copy link
Copy Markdown
Collaborator

@fpagliughi fpagliughi left a comment

Choose a reason for hiding this comment

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

LGTM

@fpagliughi fpagliughi merged commit 883d46c into socketcan-rs:master May 27, 2026
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.

2 participants