Skip to content

Issue #3459 removed check for required params#3460

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
neilotoole:issue-3459_bad_required_checking
Jul 27, 2016
Merged

Issue #3459 removed check for required params#3460
wing328 merged 1 commit intoswagger-api:masterfrom
neilotoole:issue-3459_bad_required_checking

Conversation

@neilotoole
Copy link
Copy Markdown
Contributor

This addresses #3459 by removing the check for required params.

@neilotoole neilotoole changed the title #3459 removed check for required params Issue #3459 removed check for required params Jul 27, 2016
@neilotoole
Copy link
Copy Markdown
Contributor Author

I haven't looked in detail, but it's possible the TravisCI failure was not due to the PR itself.

$ docker pull swaggerapi/petstore
Using default tag: latest
latest: Pulling from swaggerapi/petstore
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 27, 2016

@neilotoole Thanks for the PR. I've restarted the build and all tests passed. (I think the issue is with Docker Hub)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants