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.

Sign up for GitHub

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

Jump to bottom

Fea/content type #25

Merged
merged 2 commits into from Jan 4, 2017
Merged

Fea/content type #25

merged 2 commits into from Jan 4, 2017

Conversation

chadicus
Copy link
Owner

@chadicus chadicus commented Jan 4, 2017 •
edited

Fixes # .

What does this PR do?

This pull request is built on top of @achse previous PR. It adds the response Content-Type header if it does not exist.

Checklist

  • Pull request contains a clear definition of changes
  • Tests (either unit, integration, or acceptance) written and passing

@coveralls
Copy link

coveralls commented Jan 4, 2017 •
edited

Coverage Status

Coverage remained the same at 100.0% when pulling 9aa65d1 on fea/content-type into 35a097b on master.

@chadicus chadicus merged commit e2d8b6d into master Jan 4, 2017
@chadicus chadicus deleted the fea/content-type branch January 4, 2017 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@chadicus @coveralls