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

Don't unwrap header metadata #1146

Merged
merged 1 commit into from Nov 10, 2017

Conversation

Projects
None yet
6 participants
@seemyvest
Copy link
Contributor

seemyvest commented Nov 10, 2017

Fix for a comment in #1092.
r? @fitzgen

@durka

durka approved these changes Nov 10, 2017

@emilio

This comment has been minimized.

Copy link
Collaborator

emilio commented Nov 10, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Nov 10, 2017

📌 Commit 7f0aa6c has been approved by emilio

@highfive highfive assigned emilio and unassigned fitzgen Nov 10, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Nov 10, 2017

⌛️ Testing commit 7f0aa6c with merge c54e64c...

bors-servo added a commit that referenced this pull request Nov 10, 2017

Auto merge of #1146 - seemyvest:fix-metadata, r=emilio
Don't unwrap header metadata

Fix for a comment in #1092.
r? @fitzgen
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Nov 10, 2017

☀️ Test successful - status-travis
Approved by: emilio
Pushing c54e64c to master...

@bors-servo bors-servo merged commit 7f0aa6c into rust-lang:master Nov 10, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@fitzgen

This comment has been minimized.

Copy link
Member

fitzgen commented Nov 10, 2017

Thanks @seemyvest !

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.