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

Stablize {HashMap,BTreeMap}::into_{keys,values} #84328

Merged
merged 2 commits into from May 6, 2021

Conversation

Folyd
Copy link
Contributor

@Folyd Folyd commented Apr 19, 2021

I would propose to stabilize {HashMap,BTreeMap}::into_{keys,values}( aka. map_into_keys_values).

Closes #75294.

@rust-highfive
Copy link
Collaborator

r? @kennytm

(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 Apr 19, 2021
@Folyd
Copy link
Contributor Author

Folyd commented Apr 19, 2021

@rustbot modify labels: +T-libs

@rustbot rustbot added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Apr 19, 2021
@Folyd
Copy link
Contributor Author

Folyd commented Apr 19, 2021

r? @m-ou-se

@rust-highfive rust-highfive assigned m-ou-se and unassigned kennytm Apr 19, 2021
@rust-log-analyzer

This comment has been minimized.

@Folyd Folyd force-pushed the stablize_map_into_keys_values branch from 23f1611 to 33cc3f5 Compare April 19, 2021 06:23
@jonas-schievink jonas-schievink added the relnotes Marks issues that should be documented in the release notes of the next release. label Apr 19, 2021
@m-ou-se
Copy link
Member

m-ou-se commented Apr 19, 2021

We nowadays try to do the FCP on the tracking issue before the stabilization PR, to avoid starting a new discussion on the feature separate from the tracking issue. I've started the FCP process there: #75294 (comment)

@m-ou-se m-ou-se added S-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 19, 2021
@m-ou-se
Copy link
Member

m-ou-se commented Apr 24, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Apr 24, 2021

📌 Commit 33cc3f5 has been approved by m-ou-se

@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-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. labels Apr 24, 2021
@m-ou-se
Copy link
Member

m-ou-se commented Apr 24, 2021

Oh wait. FCP didn't finish yet. :)

@bors r-

@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 Apr 24, 2021
@m-ou-se m-ou-se added S-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 24, 2021
@m-ou-se
Copy link
Member

m-ou-se commented May 5, 2021

FCP finished here: #75294 (comment)

@bors r+ rollup

@bors
Copy link
Contributor

bors commented May 5, 2021

📌 Commit b6f3dbb has been approved by m-ou-se

@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-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. labels May 5, 2021
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request May 5, 2021
…, r=m-ou-se

Stablize {HashMap,BTreeMap}::into_{keys,values}

I would propose to stabilize `{HashMap,BTreeMap}::into_{keys,values}`( aka. `map_into_keys_values`).

Closes rust-lang#75294.
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request May 5, 2021
…, r=m-ou-se

Stablize {HashMap,BTreeMap}::into_{keys,values}

I would propose to stabilize `{HashMap,BTreeMap}::into_{keys,values}`( aka. `map_into_keys_values`).

Closes rust-lang#75294.
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request May 6, 2021
…, r=m-ou-se

Stablize {HashMap,BTreeMap}::into_{keys,values}

I would propose to stabilize `{HashMap,BTreeMap}::into_{keys,values}`( aka. `map_into_keys_values`).

Closes rust-lang#75294.
bors added a commit to rust-lang-ci/rust that referenced this pull request May 6, 2021
Rollup of 8 pull requests

Successful merges:

 - rust-lang#83507 (Implement RFC 2951: Native link modifiers)
 - rust-lang#84328 (Stablize {HashMap,BTreeMap}::into_{keys,values})
 - rust-lang#84712 (Simplify chdir implementation and minimize unsafe block)
 - rust-lang#84851 (:arrow_up: rust-analyzer)
 - rust-lang#84923 (Only compute Obligation `cache_key` once  in `register_obligation_at`)
 - rust-lang#84945 (E0583: Include secondary path in error message)
 - rust-lang#84949 (Fix typo in `MaybeUninit::array_assume_init` safety comment)
 - rust-lang#84950 (Revert PR 83866)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 6a6c644 into rust-lang:master May 6, 2021
@rustbot rustbot added this to the 1.54.0 milestone May 6, 2021
@jplatte jplatte mentioned this pull request May 6, 2021
65 tasks
@Folyd Folyd deleted the stablize_map_into_keys_values branch May 14, 2021 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tracking Issue for {HashMap,BTreeMap}::into_{keys,values}
8 participants