Fixing some doc comments#9646
Conversation
| #[cfg(doc)] | ||
| use bevy_hierarchy::BuildChildren; | ||
|
|
There was a problem hiding this comment.
Why remove the doc import? I find it better than repeating multiple times the full path in-line.
There was a problem hiding this comment.
The links were not working on my end.
See latest doc
There was a problem hiding this comment.
Oooh I see. For this one, this is a correct change.
| #[cfg(target_os = "android")] | ||
| pub static ANDROID_APP: std::sync::OnceLock<AndroidApp> = std::sync::OnceLock::new(); | ||
|
|
||
| /// A [`Plugin`] that uses [`winit`] to create and manage windows, and receive window and input |
There was a problem hiding this comment.
Why remove the crate-level links? I find them useful.
There was a problem hiding this comment.
in the docs the links are not displayed and look a bit weird in return.
Is this still wanted behavior?
There was a problem hiding this comment.
They work on locally generated versions of the doc (by running cargo doc --open) which is generally more useful, since the search bar can search for docs on more than a single dependency.
But I'm OK with removing them if they don't works on docs.rs
Co-authored-by: Nicola Papale <[email protected]>
alice-i-cecile
added
the
S-Ready-For-Final-Review
# Objective I've been collecting some mistakes in the documentation and fixed them --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Nicola Papale <[email protected]>
4 participants
Objective
I've been collecting some mistakes in the documentation and fixed them