Skip to content

Update cargo-fetch.md to remove cargo-prefetch reference#16568

Open
xtqqczze wants to merge 1 commit intorust-lang:masterfrom
xtqqczze:patch-1
Open

Update cargo-fetch.md to remove cargo-prefetch reference#16568
xtqqczze wants to merge 1 commit intorust-lang:masterfrom
xtqqczze:patch-1

Conversation

@xtqqczze
Copy link

Remove the mention of the cargo-prefetch plugin, as it appears to be unmaintained.

cc: @ehuss

@rustbot rustbot added the A-documenting-cargo-itself Area: Cargo's documentation label Jan 28, 2026
@rustbot rustbot added the A-cli-help Area: built-in command-line help label Jan 31, 2026
@ehuss
Copy link
Contributor

ehuss commented Feb 3, 2026

I suppose this would be fine. There never seemed to be much interest in this subcommand. However, can you say what the issue with it may be? AFAIK, it should still be working. And why this is opened as a draft?

@xtqqczze
Copy link
Author

xtqqczze commented Feb 3, 2026

I suppose this would be fine. There never seemed to be much interest in this subcommand. However, can you say what the issue with it may be? AFAIK, it should still be working. And why this is opened as a draft?

I don’t have a strong opinion, I opened this PR mainly to gather feedback. If we decide we want to keep this information, it probably belongs in the See also section rather than the Description section.

@weihanglo
Copy link
Member

it probably belongs in the See also section rather than the Description section.

kinda agreed with this. No strong opinion on removal or not

@xtqqczze
Copy link
Author

xtqqczze commented Feb 3, 2026

There never seemed to be much interest in this subcommand.

I think the bar should be high to recommend external plugins in the documentation for built-in Cargo commands. Given the low usage shown by the statistics on crates.io, I think this reference should be removed.

@xtqqczze xtqqczze marked this pull request as ready for review February 3, 2026 19:56
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 3, 2026
@rustbot
Copy link
Collaborator

rustbot commented Feb 3, 2026

r? @epage

rustbot has assigned @epage.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@epage
Copy link
Contributor

epage commented Mar 4, 2026

it probably belongs in the See also section rather than the Description section.

I was curious about the intent for the See also section. The only reference I found is from https://www.man7.org/linux/man-pages/man7/man-pages.7.html

A comma-separated list of related man pages, possibly followed by other related pages or documents.

It sounds like the intent is purely for cross-referencing but I feel that isn't as appropriate here because we would want to provide the context for why.

I think the bar should be high to recommend external plugins in the documentation for built-in Cargo commands. Given the low usage shown by the statistics on crates.io, I think this reference should be removed.

Some times the Cargo team needs to experiment and advertise the experiment to see what would be useful, so going purely off of download counts won't always be applicable.

In this case, this has been a while and ehuss mentioned it hasn't had much use. I can go either way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli-help Area: built-in command-line help A-documenting-cargo-itself Area: Cargo's documentation S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants