-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Intradoc Self::
links are "not in scope" when used in impl
blocks across modules
#93205
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.
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
T-rustdoc
Relevant to the rustdoc team, which will review and decide on the PR/issue.
Comments
camelid
added
the
A-intra-doc-links
Area: Intra-doc links, the ability to link to items in docs by name
label
Jan 25, 2022
I'll see if I can look into this. If someone else wants to work on it, feel free to claim it from me! |
camelid
added
the
T-rustdoc
Relevant to the rustdoc team, which will review and decide on the PR/issue.
label
Jan 25, 2022
This appears to have since been fixed, probably by #93805. |
Actually, it'd probably be good to add a test for this. |
camelid
added
the
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
label
Aug 9, 2022
camelid
added a commit
to camelid/rust
that referenced
this issue
Aug 10, 2022
This issue was most likely fixed by rust-lang#93805.
bors
added a commit
to rust-lang-ci/rust
that referenced
this issue
Aug 10, 2022
…mpiler-errors Rollup of 8 pull requests Successful merges: - rust-lang#100286 (Add support for link-flavor rust-lld for macOS) - rust-lang#100317 (Remove logic related to deprecated nvptx-nvidia-cuda (32-bit) target) - rust-lang#100339 (Fixes bootstrap panic when running x fmt --check ) - rust-lang#100348 (Add regression test for rust-lang#93205) - rust-lang#100349 (Refactor: remove a type string comparison) - rust-lang#100353 (Fix doc links in core::time::Duration::as_secs) - rust-lang#100359 (Special-case references to leafs in valtree pretty-printing) - rust-lang#100371 (Inline CStr::from_bytes_with_nul_unchecked::rt_impl) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
@camelid Thanks! That looks to already have trickled down into recent stable rust releases, as I can't reproduce it with |
MarijnS95
added a commit
to ash-rs/ash
that referenced
this issue
Aug 11, 2022
The report at rust-lang/rust#93205 was closed as it has presumably been fixed in rust-lang/rust#93805 which has long trickled down into stable releases, and I cannot reproduce the issue on `1.62.1` anymore (latest stable as of writing) 🎉 This workaround was originally added in #559.
Ralith
pushed a commit
to ash-rs/ash
that referenced
this issue
Aug 11, 2022
The report at rust-lang/rust#93205 was closed as it has presumably been fixed in rust-lang/rust#93805 which has long trickled down into stable releases, and I cannot reproduce the issue on `1.62.1` anymore (latest stable as of writing) 🎉 This workaround was originally added in #559.
MarijnS95
added a commit
to ash-rs/ash
that referenced
this issue
Aug 11, 2022
The report at rust-lang/rust#93205 was closed as it has presumably been fixed in rust-lang/rust#93805 which has long trickled down into stable releases, and I cannot reproduce the issue on `1.62.1` anymore (latest stable as of writing) 🎉 This workaround was originally added in #559.
MarijnS95
added a commit
to ash-rs/ash
that referenced
this issue
Aug 12, 2022
The report at rust-lang/rust#93205 was closed as it has presumably been fixed in rust-lang/rust#93805 which has long trickled down into stable releases, and I cannot reproduce the issue on `1.62.1` anymore (latest stable as of writing) 🎉 This workaround was originally added in #559.
MarijnS95
added a commit
to ash-rs/ash
that referenced
this issue
Sep 5, 2022
The report at rust-lang/rust#93205 was closed as it has presumably been fixed in rust-lang/rust#93805 which has long trickled down into stable releases, and I cannot reproduce the issue on `1.62.1` anymore (latest stable as of writing) 🎉 This workaround was originally added in #559.
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.
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
T-rustdoc
Relevant to the rustdoc team, which will review and decide on the PR/issue.
I tried this code:
I expected to see this happen:
Self::
intradoc links for theimpl
block residing inmod prelude
(whereMyNewType
is not defined) are valid links.Instead, this happened: Rustdoc thinks that
Self
, resolved toMyNewType
, is not in scope here:A workaround is to make sure that
MyNewType
, withoutsuper::
prefix, is in scope:In other words, it seems that
Self
in intradoc links resolve to the name of the item, not the full path?Meta
rustc --version --verbose
:The text was updated successfully, but these errors were encountered: