-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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 missing documentation for doc alias #90339
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
r? @camelid |
This comment has been minimized.
This comment has been minimized.
e6915c7
to
23c81a0
Compare
This comment has been minimized.
This comment has been minimized.
23c81a0
to
b229271
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some nits, otherwise looks good.
@@ -153,7 +153,7 @@ example, if you want your doctests to fail if they produce any warnings, you cou | |||
These forms of the `#[doc]` attribute are used on individual items, to control how | |||
they are documented. | |||
|
|||
## `#[doc(no_inline)]`/`#[doc(inline)]` | |||
### `#[doc(no_inline)]`/`#[doc(inline)]` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you open a followup PR to replace this with
### `#[doc(no_inline)]`/`#[doc(inline)]` | |
### `no_inline` and `inline` |
(and #[doc(hidden)]
below with hidden
) for consistency with the earlier parts of this page? In that PR, you'll also need to edit at least one error message that links to this header; it'd probably also be a good idea to add a link target for the old name, something like <span id="docno_inlinedocinline">
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
b229271
to
47786d3
Compare
📌 Commit 47786d3 has been approved by |
☀️ Test successful - checks-actions |
Finished benchmarking commit (f2707fe): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
…itles, r=camelid Unify titles in rustdoc book doc attributes chapter As discussed in rust-lang#90339. I wasn't able to find out where the link to the titles was used so let's see if the CI fails. :) r? `@camelid`
…itles, r=camelid Unify titles in rustdoc book doc attributes chapter As discussed in rust-lang#90339. I wasn't able to find out where the link to the titles was used so let's see if the CI fails. :) r? ``@camelid``
No description provided.