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

intra-doc link failed linkchecker when depending on cfg(doc) #102221

Open
scottmcm opened this issue Sep 24, 2022 · 0 comments
Open

intra-doc link failed linkchecker when depending on cfg(doc) #102221

scottmcm opened this issue Sep 24, 2022 · 0 comments
Labels
A-intra-doc-links Area: Intra-doc links, the ability to link to items in docs by name C-bug Category: This is a bug. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Comments

@scottmcm
Copy link
Member

In one version of #102169 I was using cfg(doc) to bring in a use that one of the intra-doc links needed: 96314f9

But which that worked to make x.py doc pass, it later failed linkcheck: https://github.com/rust-lang/rust/actions/runs/3111111451/jobs/5043028360

std/collections/struct.VecDeque.html:317: broken intra-doc link - [<code>mem::forget</code>]
std/collections/vec_deque/struct.VecDeque.html:317: broken intra-doc link - [<code>mem::forget</code>]

So I had to go back to a use with an allow (f0dc359) to get it to work again.

@scottmcm scottmcm added T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. A-intra-doc-links Area: Intra-doc links, the ability to link to items in docs by name labels Sep 24, 2022
@fmease fmease added the C-bug Category: This is a bug. label Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-intra-doc-links Area: Intra-doc links, the ability to link to items in docs by name C-bug Category: This is a bug. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

2 participants