Improve the docs.rs situation #17
Conversation
|
|
||
| [package.metadata.docs.rs] | ||
| all-features = true | ||
| targets = [] |
There was a problem hiding this comment.
This tells docs.rs to build the default target only. pollsterss API is the same everywhere.
See https://docs.rs/about/metadata.
| @@ -1,4 +1,5 @@ | |||
| #![doc = include_str!("../README.md")] | |||
| #![cfg_attr(docsrs, feature(doc_auto_cfg))] | |||
There was a problem hiding this comment.
This is really nice because it shows the crate feature needed to use a specific API in the Rust documentation.
See rust-lang/rust#43781.
| /// } | ||
| /// ``` | ||
| /// | ||
| /// [`pollster::block_on`]: https://docs.rs/pollster/0.3.0/pollster/fn.block_on.html |
There was a problem hiding this comment.
This is a bit annoying because it needs to be kept up-to-date with version bumps.
Initially I was thinking putting this to "latest" but this is more correct in the sense that old versions won't link to new versions ...
There was a problem hiding this comment.
That is a little annoying... Perhaps the macro crate could be made to depend on the main crate when building for docsrs, allowing us to directly link through to it?
There was a problem hiding this comment.
I tried with dev-dependencies, but rust-lang/cargo#3475.
So not any way I know.
There was a problem hiding this comment.
Ah well. I'm sure this is fine for now.
2 participants
I noticed broken links on the proc-macros and them not showing as being re-exported on docs.rs.