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 Entry::key #33148

Merged
merged 1 commit into from
Apr 30, 2016
Merged

Add Entry::key #33148

merged 1 commit into from
Apr 30, 2016

Conversation

sfackler
Copy link
Member

This method was present on both variants of Entry, but not the enum

cc #32281

r? @alexcrichton

This method was present on both variants of Entry, but not the enum

cc rust-lang#32281
@alexcrichton
Copy link
Member

Huh, seems weird! Tagging with T-libs just in case but seems like an obvious win to me.

@alexcrichton alexcrichton added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Apr 22, 2016
@alexcrichton
Copy link
Member

@bors: r+ 9e167ef

@bors
Copy link
Contributor

bors commented Apr 30, 2016

⌛ Testing commit 9e167ef with merge 46504e9...

bors added a commit that referenced this pull request Apr 30, 2016
Add Entry::key

This method was present on both variants of Entry, but not the enum

cc #32281

r? @alexcrichton
@bors
Copy link
Contributor

bors commented Apr 30, 2016

💔 Test failed - auto-win-msvc-64-opt

@sfackler
Copy link
Member Author

@bors retry

@bors bors merged commit 9e167ef into rust-lang:master Apr 30, 2016
@sfackler sfackler deleted the entry-key branch November 26, 2016 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

3 participants