-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document features automatically. #2780
Conversation
Visit the preview URL for this PR (updated for commit 3593765): https://yew-rs-api--pr2780-doc-auto-cfg-a61n4l48.web.app (expires Mon, 25 Jul 2022 04:48:17 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
Size Comparison
✅ None of the examples has changed their size significantly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice to see that this is finally possible with cargo doc
Description
This pull request enables the
doc_auto_cfg
feature which documents the required feature flags automatically.However, if an item is re-exported with
pub use
, it's not documented automatically unless the item itself has a#[doc(cfg(...))]
or#[cfg(...)]
. I have modified those to use#[cfg(...)]
instead of#[doc(cfg(...))]
.Checklist