Skip to content

fix missing serialization tag#1437

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
deads2k:deads-fix-missing-serialization
Mar 25, 2015
Merged

fix missing serialization tag#1437
openshift-bot merged 1 commit intoopenshift:masterfrom
deads2k:deads-fix-missing-serialization

Conversation

@deads2k
Copy link
Contributor

@deads2k deads2k commented Mar 24, 2015

@liggitt missing json tag and some godoc.

@deads2k deads2k force-pushed the deads-fix-missing-serialization branch from aea5362 to 52712fb Compare March 24, 2015 20:16
@deads2k deads2k mentioned this pull request Mar 25, 2015
@deads2k deads2k force-pushed the deads-fix-missing-serialization branch from 52712fb to f124a7d Compare March 25, 2015 12:13
@deads2k
Copy link
Contributor Author

deads2k commented Mar 25, 2015

@liggitt Serialization was fixed by #1440. Might still be worth the godoc, but I don't feel strongly. Let me know.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't mention keypairs... I don't think that's correct. "CertFile is a file containing a PEM-encoded certificate"

@liggitt
Copy link
Contributor

liggitt commented Mar 25, 2015

LGTM otherwise

@deads2k deads2k force-pushed the deads-fix-missing-serialization branch from f124a7d to b62263a Compare March 25, 2015 18:57
@deads2k
Copy link
Contributor Author

deads2k commented Mar 25, 2015

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/1296/) (Image: devenv-fedora_1132)

@openshift-bot
Copy link
Contributor

Evaluated for origin up to b62263a

openshift-bot pushed a commit that referenced this pull request Mar 25, 2015
@openshift-bot openshift-bot merged commit d86f9d5 into openshift:master Mar 25, 2015
@deads2k deads2k deleted the deads-fix-missing-serialization branch March 31, 2015 18:52
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