Skip to content

[Go] ParameterToString can panic#3384

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
keithballdotnet:patch-1
Jul 18, 2016
Merged

[Go] ParameterToString can panic#3384
wing328 merged 1 commit intoswagger-api:masterfrom
keithballdotnet:patch-1

Conversation

@keithballdotnet
Copy link
Copy Markdown
Contributor

Let the fmt package help convert the obj to a string. Previously this could panic if the obj was an int.

Let the fmt package help convert the obj to a string.  Previously this could panic if the obj was an int.
@keithballdotnet keithballdotnet changed the title ParameterToString can panic [Go] ParameterToString can panic Jul 15, 2016
@wing328 wing328 added this to the v2.3.0 milestone Jul 15, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 15, 2016

@keithballdotnet thanks for the PR.

cc @guohuang @neilotoole

@guohuang
Copy link
Copy Markdown
Contributor

looks good!

@keithballdotnet

@wing328 wing328 merged commit 4da6ecf into swagger-api:master Jul 18, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 18, 2016

@keithballdotnet thanks for the PR. Next time, please run ./bin/go-petstore.sh to update the Go Petstore API client as well.

@guohuang thanks for reviewing the PR.

@keithballdotnet
Copy link
Copy Markdown
Contributor Author

Sorry, @wing328 I just edited the file inline on github. Next time will do a full edit. ;)

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 18, 2016

@keithballdotnet no problem and we (the community) appreciate your contribution.

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.

3 participants