github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-31 15:25:57 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #17703 from timstclair/api-conventions

Browse Source 
Auto commit by PR queue bot
This commit is contained in:
k8s-merge-robot 2015-11-30 15:49:53 -08:00
commit 59a89a5fd9
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
docs/devel/api-conventions.md
View File

@ -387,7 +387,8 @@ Fields must be either optional or required.
Optional fields have the following properties:
- They have `omitempty` struct tag in Go.
- They are a pointer type in the Go definition (e.g. `bool *awesomeFlag`).
- They are a pointer type in the Go definition (e.g. `bool *awesomeFlag`) or have a built-in `nil`
value (e.g. maps and slices).
- The API server should allow POSTing and PUTing a resource with this field unset.
Required fields have the opposite properties, namely:
@ -409,7 +410,8 @@ codebase. However:
- having a pointer consistently imply optional is clearer for users of the Go language client, and any
other clients that use corresponding types
Therefore, we ask that pointers always be used with optional fields.
Therefore, we ask that pointers always be used with optional fields that do not have a built-in
`nil` value.
## Defaulting