-
Notifications
You must be signed in to change notification settings - Fork 54
Prototype: Add defaulter support #27
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
Conversation
I haven't forgotten this I've just been very busy with work stuff (ironically: related to kubernetes) and haven't have been able to properly grok this PR. |
could you update this? |
@casualjim just returned from vacation today. Will update after having catched up with mails. |
@casualjim rebased and simplified a lot |
Codecov Report
@@ Coverage Diff @@
## master #27 +/- ##
==========================================
+ Coverage 83.71% 84.03% +0.31%
==========================================
Files 10 10
Lines 1363 1384 +21
==========================================
+ Hits 1141 1163 +22
+ Misses 182 180 -2
- Partials 40 41 +1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #27 +/- ##
==========================================
+ Coverage 83.71% 84.03% +0.31%
==========================================
Files 10 10
Lines 1363 1384 +21
==========================================
+ Hits 1141 1163 +22
+ Misses 182 180 -2
- Partials 40 41 +1
Continue to review full report at Codecov.
|
We plan to use go-openapi for CustomResourceDefinition validation in Kubernetes (compare kubernetes/community#708). For that will need support for default values. This is a prototype to integrate this into go-openapi.