Skip to content

update ty::TyKind documentation #93781

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Feb 9, 2022
Merged

update ty::TyKind documentation #93781

merged 3 commits into from
Feb 9, 2022

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented Feb 8, 2022

slightly unsure about ty::Opaque and ty::Bound/ty::Placeholder.

r? @jackh726 @nikomatsakis @oli-obk

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Feb 8, 2022
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 8, 2022
lcnr
lcnr commented Feb 8, 2022
View reviewed changes
lcnr
lcnr commented Feb 8, 2022
View reviewed changes
oli-obk
oli-obk reviewed Feb 8, 2022
View reviewed changes
@nikomatsakis
Copy link
Contributor

Thanks for doing this, @lcnr, these kinds of things are always helpful.

@lcnr @oli-obk
Update compiler/rustc_middle/src/ty/sty.rs
af77bdf 
Co-authored-by: Oli Scherer <github35764891676564198441@oli-obk.de>
lcnr
lcnr commented Feb 8, 2022
View reviewed changes
@jackh726
Copy link
Member

jackh726 commented Feb 8, 2022

This is fantastic. Thanks!

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 8, 2022

📌 Commit a8be000 has been approved by jackh726

@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 Feb 8, 2022
@jackh726
Copy link
Member

jackh726 commented Feb 8, 2022

@bors rollup

@JohnTitor JohnTitor mentioned this pull request Feb 9, 2022
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 9, 2022
@bors
Auto merge of rust-lang#93795 - JohnTitor:rollup-n0dmsoo, r=JohnTitor
bf242bb 
Rollup of 7 pull requests

Successful merges:

 - rust-lang#93445 (Add From<u8> for ExitCode)
 - rust-lang#93694 (rustdoc: tweak line spacing and paragraph spacing for accessibility)
 - rust-lang#93735 (Stabilize int_abs_diff in 1.60.0.)
 - rust-lang#93746 (Remove defaultness from ImplItem.)
 - rust-lang#93748 (rustc_query_impl: reduce visibility of some modules/fn's)
 - rust-lang#93751 (Drop tracking: track borrows of projections)
 - rust-lang#93781 (update `ty::TyKind` documentation)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 7f4486b into rust-lang:master Feb 9, 2022
@rustbot rustbot added this to the 1.60.0 milestone Feb 9, 2022
@lcnr lcnr deleted the ty-kind-docs branch February 9, 2022 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikomatsakis nikomatsakis nikomatsakis left review comments

@oli-obk oli-obk oli-obk left review comments

Assignees

@jackh726 jackh726

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.60.0
Development

Successfully merging this pull request may close these issues.
7 participants
@lcnr @nikomatsakis @jackh726 @bors @oli-obk @rust-highfive @rustbot