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

Fixed mismatched var names (consumes vs contentType) #108

Merged
merged 1 commit into from
Aug 6, 2014

Conversation

unsetbit
Copy link
Contributor

@unsetbit unsetbit commented Aug 4, 2014

'consumes' was not defined in the scope but used, whereas 'contentType' was defined but not used.

Consumes was not defined in the scope but use, where as content-
Type was defined but not used.
@unsetbit
Copy link
Contributor Author

unsetbit commented Aug 4, 2014

Also, the unit tests seem to require another local server running (I see it trying and failint to hit localhost:8002), but I don't see how/what to set that up in the docs, so I was unable to run the tests against this change.

fehguy added a commit that referenced this pull request Aug 6, 2014
Fixed mismatched var names (consumes vs contentType)
@fehguy fehguy merged commit 140e912 into swagger-api:master Aug 6, 2014
@fehguy
Copy link
Contributor

fehguy commented Aug 6, 2014

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.

2 participants