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

rustdoc: show implementations on #[fundamental] wrappers #96565

Merged
merged 1 commit into from
May 20, 2022

Conversation

notriddle
Copy link
Contributor

Fixes #92940

@rustbot rustbot added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Apr 29, 2022
@rust-highfive
Copy link
Collaborator

r? @CraftSpider

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 29, 2022
@notriddle
Copy link
Contributor Author

r? @camelid

@rust-highfive rust-highfive assigned camelid and unassigned CraftSpider May 16, 2022
Copy link
Member

@camelid camelid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your explanation at #92940 (comment). It made the reviewing much easier.

@camelid
Copy link
Member

camelid commented May 20, 2022

@bors r+ rollup

@bors
Copy link
Contributor

bors commented May 20, 2022

📌 Commit 62b9e06 has been approved by camelid

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 20, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request May 20, 2022
…laumeGomez

Rollup of 6 pull requests

Successful merges:

 - rust-lang#96565 (rustdoc: show implementations on `#[fundamental]` wrappers)
 - rust-lang#97179 (Add new lint to enforce whitespace after keywords)
 - rust-lang#97185 (interpret/validity: separately control checking numbers for being init and non-ptr)
 - rust-lang#97188 (Remove unneeded null pointer asserts in ptr2int casts)
 - rust-lang#97189 (Update .mailmap)
 - rust-lang#97192 (Say "last" instead of "rightmost" in the documentation for `std::str:rfind`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit afcf099 into rust-lang:master May 20, 2022
@rustbot rustbot added this to the 1.63.0 milestone May 20, 2022
@notriddle notriddle deleted the notriddle/impl-box branch May 20, 2022 15:44
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 8, 2022
…t-of-fundamental, r=notriddle

rustdoc: Document effect of fundamental types

This was implemented in rust-lang#96565, but not documented. But it's a useful feature for everyone who implements own wrapper (especially wrappers-around-pointers) types, so that they can behave like pointers (and stdlib wrappers) do -- so here goes a mention in the unstable section of the rustdoc book.

(That is where I initially looked to find tricks for making my own wrapper types be transparent to documentation).
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 8, 2022
…t-of-fundamental, r=notriddle

rustdoc: Document effect of fundamental types

This was implemented in rust-lang#96565, but not documented. But it's a useful feature for everyone who implements own wrapper (especially wrappers-around-pointers) types, so that they can behave like pointers (and stdlib wrappers) do -- so here goes a mention in the unstable section of the rustdoc book.

(That is where I initially looked to find tricks for making my own wrapper types be transparent to documentation).
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Oct 8, 2022
…t-of-fundamental, r=notriddle

rustdoc: Document effect of fundamental types

This was implemented in rust-lang#96565, but not documented. But it's a useful feature for everyone who implements own wrapper (especially wrappers-around-pointers) types, so that they can behave like pointers (and stdlib wrappers) do -- so here goes a mention in the unstable section of the rustdoc book.

(That is where I initially looked to find tricks for making my own wrapper types be transparent to documentation).
notriddle added a commit to notriddle/rust that referenced this pull request Oct 9, 2022
…t-of-fundamental, r=notriddle

rustdoc: Document effect of fundamental types

This was implemented in rust-lang#96565, but not documented. But it's a useful feature for everyone who implements own wrapper (especially wrappers-around-pointers) types, so that they can behave like pointers (and stdlib wrappers) do -- so here goes a mention in the unstable section of the rustdoc book.

(That is where I initially looked to find tricks for making my own wrapper types be transparent to documentation).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rustdoc: Impl for boxed type doesn't show up on the type's page
6 participants