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

Hunted down and deleted all remaining deprecated owned vectors from docs #14553

Merged
merged 5 commits into from May 31, 2014

Conversation

Projects
None yet
3 participants
@reem
Copy link
Contributor

commented May 31, 2014

I removed all remaining deprecated owned vectors from the docs. All example tests pass.

@@ -30,7 +30,6 @@ let x: int = 42;
let y: String = format!("{:t}", x); // binary
let y: String = format!("{:o}", x); // octal
let y: String = format!("{:x}", x); // lowercase hexadecimal
let y: String = format!("{:X}", x); // uppercase hexadecimal

This comment has been minimized.

Copy link
@alexcrichton

alexcrichton May 31, 2014

Member

Why was this deleted?

This comment has been minimized.

Copy link
@reem

reem May 31, 2014

Author Contributor

Oh. I realize what happened. I accidentally moved this line using vim which randomly broke some other test that I then thought was broken already. Give me a moment and I'll fix it.

@bors

This comment has been minimized.

Copy link
Contributor

commented on 1959925 May 31, 2014

saw approval from alexcrichton
at reem@1959925

This comment has been minimized.

Copy link
Contributor

replied May 31, 2014

merging reem/rust/nuke-owned-vectors = 1959925 into auto

This comment has been minimized.

Copy link
Contributor

replied May 31, 2014

reem/rust/nuke-owned-vectors = 1959925 merged ok, testing candidate = faa7ba7

This comment has been minimized.

Copy link
Contributor

replied May 31, 2014

fast-forwarding master to auto = faa7ba7

bors added a commit that referenced this pull request May 31, 2014

auto merge of #14553 : reem/rust/nuke-owned-vectors, r=alexcrichton
I removed all remaining deprecated owned vectors from the docs. All example tests pass.

@bors bors closed this May 31, 2014

@bors bors merged commit 1959925 into rust-lang:master May 31, 2014

2 checks passed

continuous-integration/travis-ci The Travis CI build passed
Details
default all tests passed

@reem reem deleted the reem:nuke-owned-vectors branch Jun 2, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.