Skip to content

Unstabilize Vec::remove_item #68089

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 12, 2020
Merged

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Jan 10, 2020

As concerned by @kornelski, @LukasKalbertodt, and @gnzlbg in #40062.
Reverts #67727

@sfackler
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 10, 2020

📌 Commit cecfdc1c826fae192128dbd737580d6bf59afc74 has been approved by sfackler

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 10, 2020
bors added a commit that referenced this pull request Jan 10, 2020
Rollup of 8 pull requests

Successful merges:

 - #66045 (Add method Result::into_ok)
 - #67258 (Introduce `X..`, `..X`, and `..=X` range patterns)
 - #68014 (Unify output of "variant not found" errors)
 - #68019 (Build compiletest with in-tree libtest)
 - #68039 (remove explicit strip-hidden pass from compiler doc generation)
 - #68050 (Canonicalize rustc_error imports)
 - #68059 (Allow specifying LLVM args in target specifications)
 - #68075 (rustbuild: Cleanup book generation)

Failed merges:

 - #68089 (Unstabilize `Vec::remove_item`)

r? @ghost
@Centril
Copy link
Contributor

Centril commented Jan 10, 2020

For future reference, when you unstabilize something, please also remove the stabilizing PR from relnotes and the milestone.

@bors
Copy link
Collaborator

bors commented Jan 10, 2020

☔ The latest upstream changes (presumably #68101) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 10, 2020
…item, r=alexcrichton"

This reverts commit 4ed415b, reversing
changes made to 3cce950.
@tesuji tesuji force-pushed the revert-remote_item branch from cecfdc1 to 7ba25ac Compare January 11, 2020 03:05
@tesuji
Copy link
Contributor Author

tesuji commented Jan 11, 2020

Rebased.

@Dylan-DPC-zz
Copy link

@bors r=sfackler rollup

@bors
Copy link
Collaborator

bors commented Jan 11, 2020

📌 Commit 7ba25ac has been approved by sfackler

@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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 11, 2020
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jan 11, 2020
Centril added a commit to Centril/rust that referenced this pull request Jan 12, 2020
bors added a commit that referenced this pull request Jan 12, 2020
Rollup of 6 pull requests

Successful merges:

 - #67494 (Constify more of alloc::Layout)
 - #67867 (Correctly check for opaque types in `assoc_ty_def`)
 - #67948 (Galloping search for binary_search_util)
 - #68045 (Move more of `rustc::lint` into `rustc_lint`)
 - #68089 (Unstabilize `Vec::remove_item`)
 - #68108 (Add suggestions when encountering chained comparisons)

Failed merges:

r? @ghost
@bors bors merged commit 7ba25ac into rust-lang:master Jan 12, 2020
@tesuji tesuji deleted the revert-remote_item branch January 12, 2020 06:26
# for free to join this conversation on GitHub. Already have an account? # 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