Commit c59fad14 authored by k8s-merge-robot's avatar k8s-merge-robot

Merge pull request #18941 from smarterclayton/document_typed_ints

Auto commit by PR queue bot
parents 29754318 a79dc07e
...@@ -256,6 +256,7 @@ This rule maintains the invariant that all JSON/YAML keys are fields in API obje ...@@ -256,6 +256,7 @@ This rule maintains the invariant that all JSON/YAML keys are fields in API obje
* Do not use unsigned integers, due to inconsistent support across languages and libraries. Just validate that the integer is non-negative if that's the case. * Do not use unsigned integers, due to inconsistent support across languages and libraries. Just validate that the integer is non-negative if that's the case.
* Do not use enums. Use aliases for string instead (e.g., `NodeConditionType`). * Do not use enums. Use aliases for string instead (e.g., `NodeConditionType`).
* Look at similar fields in the API (e.g., ports, durations) and follow the conventions of existing fields. * Look at similar fields in the API (e.g., ports, durations) and follow the conventions of existing fields.
* All public integer fields MUST use the Go `(u)int32` or Go `(u)int64` types, not `(u)int` (which is ambiguous depending on target platform). Internal types may use `(u)int`.
#### Constants #### Constants
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment