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

add some docs to From conversions #51474

Merged
merged 1 commit into from
Jun 10, 2018
Merged

add some docs to From conversions #51474

merged 1 commit into from
Jun 10, 2018

Conversation

llogiq
Copy link
Contributor

@llogiq llogiq commented Jun 10, 2018

This adds a helpful document to the bool → int* conversions as well as to the lossless integer conversions.

One of #51430 down, some more to go.

@rust-highfive
Copy link
Collaborator

r? @TimNN

(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 Jun 10, 2018
@TimNN
Copy link
Contributor

TimNN commented Jun 10, 2018

Did you build the docs locally to verify that the stringify! / concat! works as expected?

If so R=me.

@llogiq
Copy link
Contributor Author

llogiq commented Jun 10, 2018

Actually my build failed because I had activated --debuginfo-lines and it couldn't find gdb for some reason. I think I've now fixed that issues and am resuming the build now.

@llogiq
Copy link
Contributor Author

llogiq commented Jun 10, 2018

docs built successfully.

@bors r=TimNN

@bors
Copy link
Contributor

bors commented Jun 10, 2018

@llogiq: 🔑 Insufficient privileges: Not in reviewers

@Mark-Simulacrum
Copy link
Member

@bors r=TimNN

@bors
Copy link
Contributor

bors commented Jun 10, 2018

📌 Commit 553a44a has been approved by TimNN

@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 Jun 10, 2018
@bors
Copy link
Contributor

bors commented Jun 10, 2018

⌛ Testing commit 553a44a with merge cabb679...

bors added a commit that referenced this pull request Jun 10, 2018
add some docs to `From` conversions

This adds a helpful document to the bool → int* conversions as well as to the lossless integer conversions.

One of #51430 down, some more to go.
@bors
Copy link
Contributor

bors commented Jun 10, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: TimNN
Pushing cabb679 to master...

@bors bors merged commit 553a44a into rust-lang:master Jun 10, 2018
@llogiq llogiq deleted the from-docs branch June 10, 2018 21:10
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants