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

Clarify try! doc example #32893

Merged
merged 1 commit into from
Apr 15, 2016
Merged

Clarify try! doc example #32893

merged 1 commit into from
Apr 15, 2016

Conversation

khernyo
Copy link
Contributor

@khernyo khernyo commented Apr 11, 2016

The original is correct, but a bit misleading.

r? @steveklabnik

The original is correct, but a bit misleading.
@steveklabnik
Copy link
Member

@bors: r+ rollup

thanks!

@bors
Copy link
Contributor

bors commented Apr 12, 2016

📌 Commit 0c8a7f2 has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Apr 14, 2016
…bnik

Clarify try! doc example

The original is correct, but a bit misleading.

r? @steveklabnik
@bors
Copy link
Contributor

bors commented Apr 14, 2016

⌛ Testing commit 0c8a7f2 with merge 46304cd...

@bors
Copy link
Contributor

bors commented Apr 14, 2016

💔 Test failed - auto-win-gnu-32-opt

@alexcrichton
Copy link
Member

@bors: retry

On Thu, Apr 14, 2016 at 12:54 PM, bors notifications@github.com wrote:

[image: 💔] Test failed - auto-win-gnu-32-opt
http://buildbot.rust-lang.org/builders/auto-win-gnu-32-opt/builds/3804


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#32893 (comment)

bors added a commit that referenced this pull request Apr 14, 2016
@bors bors merged commit 0c8a7f2 into rust-lang:master Apr 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants