Skip to content
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

Add documentation for docs.rs metadata #7164

Closed
Stargateur opened this issue Jul 23, 2019 · 4 comments
Closed

Add documentation for docs.rs metadata #7164

Stargateur opened this issue Jul 23, 2019 · 4 comments
Labels
A-documenting-cargo-itself Area: Cargo's documentation S-needs-mentor Status: Issue or feature is accepted, but needs a team member to commit to helping and reviewing.

Comments

@Stargateur
Copy link

A question pop on stackoverflow today asking about how one could build the doc when build.rs fail because there is not the lib needed to link. The lib should not be needed for a doc build so the question ask about how we could skip the build script somehow.

The solution exist but it's very poorly documented, in the about page ! And https://doc.rust-lang.org/cargo/commands/cargo-metadata.html or https://doc.rust-lang.org/cargo/reference/manifest.html#package-metadata doesn't do better. Would be nice if someone who write better English than me write some doc for it.

Note: I'm surprise this go behind metadata, in my opinion metadata should not affect build, well, this too could be clarify.

rust-lang/rust-playground#192

@Stargateur Stargateur added the C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` label Jul 23, 2019
@ehuss ehuss added A-documenting-cargo-itself Area: Cargo's documentation and removed C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` labels Aug 20, 2019
@ehuss ehuss changed the title Add documentation for features doc Add documentation for docs.rs metadata Dec 19, 2019
@epage
Copy link
Contributor

epage commented Apr 21, 2022

What do people think of cross-linking the metadata table docs to https://docs.rs/about/metadata?

The main concern I can think of is the slippery slope of what all should be cross-linked. I'd limit it to official Rust projects, like docs.rs, the line distinguishing the individual parts is blurred. When more explicitly dealing with an external process, like cargo-releases package.metadata.release, its more obvious that where to look for the documentation.

@Stargateur
Copy link
Author

wow 3 years It feel it someone else that open this issue. I updated the SO question with what docs.io advice in 2022 so now maybe add a link on build page that link to https://docs.rs/about/builds is better.

But metadata page of cargo could also use a link to the metadata table of docs.io

@ehuss
Copy link
Contributor

ehuss commented Apr 21, 2022

I think it would be great to link to the documentation of docs.rs. Linking to other rust-lang projects is fine.

I know for myself, it still takes me a while to find the docs.rs docs, and I've seen them many times. They do seem to be somewhat hidden.

I'm not sure exactly where to link it. As mentioned, there are multiple points of interaction (metadata table, cargo doc, build scripts, publishing, etc.), and I'd be cautious to emphasize it in too many places. Sorry that isn't too constructive. I'd suggest picking a couple different places where you think it seems like it would be likely to be discovered and found.

@weihanglo weihanglo added S-accepted Status: Issue or feature is accepted, and has a team member available to help mentor or review S-needs-mentor Status: Issue or feature is accepted, but needs a team member to commit to helping and reviewing. and removed S-accepted Status: Issue or feature is accepted, and has a team member available to help mentor or review labels May 13, 2023
epage added a commit to epage/cargo that referenced this issue May 24, 2023
This gets us part of the cross-linking described in rust-lang#7164
epage added a commit to epage/cargo that referenced this issue Oct 25, 2023
This gets us part of the cross-linking described in rust-lang#7164
epage added a commit to epage/cargo that referenced this issue Oct 25, 2023
This gets us part of the cross-linking described in rust-lang#7164
bors added a commit that referenced this issue Oct 25, 2023
docs(ref): Link to docs.rs metadata table

This gets us part of the cross-linking described in #7164
Rustin170506 pushed a commit to Rustin170506/cargo that referenced this issue Oct 27, 2023
This gets us part of the cross-linking described in rust-lang#7164
@epage
Copy link
Contributor

epage commented Nov 1, 2023

Probably the link we added in #12879 is sufficient.

@epage epage closed this as completed Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-documenting-cargo-itself Area: Cargo's documentation S-needs-mentor Status: Issue or feature is accepted, but needs a team member to commit to helping and reviewing.
Projects
None yet
Development

No branches or pull requests

4 participants