Stabilize vec_try_remove - #163459
Stabilize vec_try_remove#163459
Conversation
|
Thanks for the pull request, and welcome! The Rust Project has assigned @jhpratt (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks. Please see the contribution instructions and our LLM policy for more information. Why was this reviewer chosen?The reviewer was selected based on:
|
There was a problem hiding this comment.
Welcome, and thanks for the PR! We hope you'll stick around and make a positive impact on Rust. Feel free to reach out on Zulip if you ever get stuck.
This stabilization PR looks like it's been prepared correctly; it just needs approval from a contributor with r+ permissions, likely @jhpratt. It's normal for this to take a while, as there are a lot of PRs to review.
Or r=me once tests pass (I don't have r+ perms yet but I'm aiming to earn them)
Disclaimer: I am a coworker of Petra's at Canonical.
|
Welcome and thanks for the PR from my side, too :) r? me @bors r+ rollup cc @rust-lang/libs for awareness (FCP was in July, predating the libs team refactor and FCPs on stabilization PRs) |
|
FWIW we do have @ rust-lang/libs-ping as well as a way of maybe reducing the number of folks who are pinged. (not pinging a second time, though) |
|
Yeah, I remembered that right after I sent out the comment 😅 I adapted the message from the last time I did this, when we didn’t have that ping group yet. |
…, r=hanna-kruppe Stabilize vec_try_remove Closes rust-lang#146954 , which is has [completed its FCP with disposition to merge](rust-lang#146954 (comment)). Hello, this is my first contribution to Rust! I was programming for fun this weekend and I was surprised that this wasn't stabilized, so I thought I would try to contribute. Disclosure: I am making this PR in my capacity as a Canonical employee. No LLMs were harmed in the making of this pull request.
Rollup of 9 pull requests Successful merges: - #161015 (Stabilize `funnel_shifts` (including `const`)) - #161712 (Stabilize `Result::into_{ok,err}`) - #162493 (Add support for -Zsanitizer-cfi-minimal-runtime) - #163427 (implement #![feature(gca_adts)]) - #163390 (add `automatically_derived` attribute documentation) - #163428 (do not complain about unstable target features on nightly) - #163444 (Add `stable_rustc` helper in `run-make-support`) - #163447 (Allow using different index types when reading and writing to tables) - #163459 (Stabilize vec_try_remove)
Rollup of 9 pull requests Successful merges: - rust-lang/rust#161015 (Stabilize `funnel_shifts` (including `const`)) - rust-lang/rust#161712 (Stabilize `Result::into_{ok,err}`) - rust-lang/rust#162493 (Add support for -Zsanitizer-cfi-minimal-runtime) - rust-lang/rust#163427 (implement #![feature(gca_adts)]) - rust-lang/rust#163390 (add `automatically_derived` attribute documentation) - rust-lang/rust#163428 (do not complain about unstable target features on nightly) - rust-lang/rust#163444 (Add `stable_rustc` helper in `run-make-support`) - rust-lang/rust#163447 (Allow using different index types when reading and writing to tables) - rust-lang/rust#163459 (Stabilize vec_try_remove)
Rollup of 9 pull requests Successful merges: - rust-lang/rust#161015 (Stabilize `funnel_shifts` (including `const`)) - rust-lang/rust#161712 (Stabilize `Result::into_{ok,err}`) - rust-lang/rust#162493 (Add support for -Zsanitizer-cfi-minimal-runtime) - rust-lang/rust#163427 (implement #![feature(gca_adts)]) - rust-lang/rust#163390 (add `automatically_derived` attribute documentation) - rust-lang/rust#163428 (do not complain about unstable target features on nightly) - rust-lang/rust#163444 (Add `stable_rustc` helper in `run-make-support`) - rust-lang/rust#163447 (Allow using different index types when reading and writing to tables) - rust-lang/rust#163459 (Stabilize vec_try_remove)
Rollup of 9 pull requests Successful merges: - rust-lang/rust#161015 (Stabilize `funnel_shifts` (including `const`)) - rust-lang/rust#161712 (Stabilize `Result::into_{ok,err}`) - rust-lang/rust#162493 (Add support for -Zsanitizer-cfi-minimal-runtime) - rust-lang/rust#163427 (implement #![feature(gca_adts)]) - rust-lang/rust#163390 (add `automatically_derived` attribute documentation) - rust-lang/rust#163428 (do not complain about unstable target features on nightly) - rust-lang/rust#163444 (Add `stable_rustc` helper in `run-make-support`) - rust-lang/rust#163447 (Allow using different index types when reading and writing to tables) - rust-lang/rust#163459 (Stabilize vec_try_remove)
Closes #146954 , which is has completed its FCP with disposition to merge.
Hello, this is my first contribution to Rust! I was programming for fun this weekend and I was surprised that this wasn't stabilized, so I thought I would try to contribute.
Disclosure: I am making this PR in my capacity as a Canonical employee.
No LLMs were harmed in the making of this pull request.