ai nits

mjudeikis · mjudeikis · commit 5240895e593c · 2025-04-09T17:20:14.000+03:00
diff --git a/pkg/admission/workspace/admission.go b/pkg/admission/workspace/admission.go
@@ -213,7 +213,7 @@ func (o *workspace) Validate(ctx context.Context, a admission.Attributes, _ admi
 				return admission.NewForbidden(a, errors.New("spec.type cannot be set for mounted workspaces"))
 			}
 			// Check for immutability of spec.type via pointers checks first.
-			if !isSystemPrivileged && (old.Spec.Type == nil && ws.Spec.Type != nil) || (old.Spec.Type != nil && ws.Spec.Type == nil) {
+			if !isSystemPrivileged && ((old.Spec.Type == nil && ws.Spec.Type != nil) || (old.Spec.Type != nil && ws.Spec.Type == nil)) {
 				return admission.NewForbidden(a, errors.New("spec.type is immutable"))
 			}
 			// Check for immutability of spec.type via field checks.

Original file line number	Diff line number	Diff line change
`@@ -213,7 +213,7 @@ func (o *workspace) Validate(ctx context.Context, a admission.Attributes, _ admi`
`213`	`213`	`return admission.NewForbidden(a, errors.New("spec.type cannot be set for mounted workspaces"))`
`214`	`214`	`}`
`215`	`215`	`// Check for immutability of spec.type via pointers checks first.`
`216`		`- if !isSystemPrivileged && (old.Spec.Type == nil && ws.Spec.Type != nil) \|\| (old.Spec.Type != nil && ws.Spec.Type == nil) {`
	`216`	`+ if !isSystemPrivileged && ((old.Spec.Type == nil && ws.Spec.Type != nil) \|\| (old.Spec.Type != nil && ws.Spec.Type == nil)) {`
`217`	`217`	`return admission.NewForbidden(a, errors.New("spec.type is immutable"))`
`218`	`218`	`}`
`219`	`219`	`// Check for immutability of spec.type via field checks.`