Skip to content

Conversation

rap2hpoutre
Copy link
Contributor

This PR is a part of #29378. I submit this PR with the help (mentoring) of @steveklabnik. I'm still not sure my request is good enough but I don't want to spoil the issue with too much questions so I continue here. r? @steveklabnik

@shepmaster
Copy link
Member

Thanks for the PR, @rap2hpoutre! We'll make sure to have @steveklabnik or another member of the team review this in a short amount of time!

@shepmaster shepmaster added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 5, 2017
@frewsxcv
Copy link
Contributor

frewsxcv commented May 6, 2017

Thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented May 6, 2017

📌 Commit 71aaab1 has been approved by frewsxcv

@frewsxcv frewsxcv assigned frewsxcv and unassigned steveklabnik May 6, 2017
@bors
Copy link
Collaborator

bors commented May 6, 2017

⌛ Testing commit 71aaab1 with merge 8aad3a3...

bors added a commit that referenced this pull request May 6, 2017
Add an example to std::thread::Result type

This PR is a part of #29378. I submit this PR with the help (mentoring) of @steveklabnik. I'm still not sure my request is good enough but I don't want to spoil the issue with too much questions so I continue here. r? @steveklabnik
@bors
Copy link
Collaborator

bors commented May 6, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: frewsxcv
Pushing 8aad3a3 to master...

@bors bors merged commit 71aaab1 into rust-lang:master May 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants