-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
The logo on docs.rs/defmt is missing #904
Comments
I am pretty sure the logo was shown before, so I think it might be a problem on the docs.rs side as well. |
It is not docs.rs fault, but because the logo was fetched from the knurling website which not available anymore. |
Instead of relying on an external link the image should be included locally.
|
After lots of searching, and reading this issue: rust-lang/rust#32104, I think using local images isn't possible / simple, and even by providing the url of the logo in the book folder I couldn't manage to render it with |
For me the logo shows when changing the link: diff --git a/defmt/src/lib.rs b/defmt/src/lib.rs
index 605e96d..d4d4055 100644
--- a/defmt/src/lib.rs
+++ b/defmt/src/lib.rs
@@ -17,7 +17,7 @@
#![cfg_attr(not(feature = "unstable-test"), no_std)]
// NOTE if you change this URL you'll also need to update all other crates in this repo
-#![doc(html_logo_url = "https://knurling.ferrous-systems.com/knurling_logo_light_text.svg")]
+#![doc(html_logo_url = "https://defmt.ferrous-systems.com/assets/knurling_logo_light_text.svg")]
#![warn(missing_docs)]
#[cfg(feature = "alloc")] |
You just get a missing-image box at docs.rs/defmt.
The text was updated successfully, but these errors were encountered: