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

docs(headers): move blank line in right place. #611

Merged
merged 1 commit into from
Jul 23, 2015

Conversation

batisteo
Copy link

@GitCop
Copy link

GitCop commented Jul 15, 2015

Thanks for contributing! Unfortunately, I'm here to tell you there were the following style issues with your Pull Request:

  • Commit: 65ba136
    • Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md


This message was auto-generated by https://gitcop.com

@batisteo batisteo changed the title Doc: moved blank line in right place. docs(headers): move blank line in right place. Jul 16, 2015
@batisteo
Copy link
Author

Changed from:

use hyper::header::{Headers, ContentType};
use hyper::mime::{Mime, TopLevel, SubLevel, Attr, Value};

let mut headers = Headers::new();
headers.set(
    ContentType(Mime(TopLevel::Application, SubLevel::Json,
                     vec![(Attr::Charset, Value::Utf8)]))

);

to:

use hyper::header::{Headers, ContentType};
use hyper::mime::{Mime, TopLevel, SubLevel, Attr, Value};

let mut headers = Headers::new();

headers.set(
    ContentType(Mime(TopLevel::Application, SubLevel::Json,
                     vec![(Attr::Charset, Value::Utf8)]))
);

for consistency with previous block.

seanmonstar added a commit that referenced this pull request Jul 23, 2015
docs(headers): move blank line in right place.
@seanmonstar seanmonstar merged commit 3bd0635 into hyperium:master Jul 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants