Skip to content

Commit d866750

Browse files
kflynnk8s-ci-robot
authored andcommitted
chore: Review feedback: Drop empty string from doc comments
Signed-off-by: Flynn <[email protected]>
1 parent 84a8806 commit d866750

12 files changed

+72
-88
lines changed

apis/v1/gateway_types.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -299,11 +299,10 @@ type GatewaySpec struct {
299299
// meaning it will dynamically and implicitly have Routes (e.g. HTTPRoute)
300300
// attached to it, according to the scope configured here.
301301
//
302-
// If unset (the default), set to an empty string (`""`), or set to None,
303-
// the Gateway will not act as a default Gateway; if set, the Gateway will
304-
// claim any Route with a matching scope set in its UseDefaultGateway
305-
// field, subject to the usual rules about which routes the Gateway can
306-
// attach to.
302+
// If unset (the default) or set to None, the Gateway will not act as a
303+
// default Gateway; if set, the Gateway will claim any Route with a
304+
// matching scope set in its UseDefaultGateway field, subject to the usual
305+
// rules about which routes the Gateway can attach to.
307306
//
308307
// Think carefully before using this functionality! While the normal rules
309308
// about which Route can apply are still enforced, it is simply easier for

apis/v1/shared_types.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -248,11 +248,10 @@ type CommonRouteSpec struct {
248248
ParentRefs []ParentReference `json:"parentRefs,omitempty"`
249249

250250
// UseDefaultGateways indicates the default Gateway scope to use for this
251-
// Route. If unset (the default), set to an empty string (`""`), or set to
252-
// None, the Route will not be attached to any default Gateway; if set, it
253-
// will be attached to any default Gateway supporting the named scope,
254-
// subject to the usual rules about which Routes a Gateway is allowed to
255-
// claim.
251+
// Route. If unset (the default) or set to None, the Route will not be
252+
// attached to any default Gateway; if set, it will be attached to any
253+
// default Gateway supporting the named scope, subject to the usual rules
254+
// about which Routes a Gateway is allowed to claim.
256255
//
257256
// Think carefully before using this functionality! The set of default
258257
// Gateways supporting the requested scope can change over time without

config/crd/experimental/gateway.networking.k8s.io_gateways.yaml

Lines changed: 8 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml

Lines changed: 4 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml

Lines changed: 8 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_tcproutes.yaml

Lines changed: 4 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml

Lines changed: 8 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_udproutes.yaml

Lines changed: 4 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/standard/gateway.networking.k8s.io_gateways.yaml

Lines changed: 8 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/standard/gateway.networking.k8s.io_grpcroutes.yaml

Lines changed: 4 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)