Skip to content

Commit c1a7c99

Browse files
committed
refactor: simplify V1Patch constructor and improve argument validation
1 parent f695e4b commit c1a7c99

File tree

2 files changed

+5
-10
lines changed

2 files changed

+5
-10
lines changed

src/KubernetesClient/Models/V1Patch.cs

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,13 @@ public enum PatchType
3939

4040
public V1Patch(object body, PatchType type)
4141
{
42-
Content = body;
43-
Type = type;
44-
45-
if (Content == null)
42+
if (type == PatchType.Unknown)
4643
{
47-
throw new ArgumentNullException(nameof(Content), "object must be set");
44+
throw new ArgumentException("patch type must be set", nameof(type));
4845
}
4946

50-
if (Type == PatchType.Unknown)
51-
{
52-
throw new ArgumentException("patch type must be set", nameof(Type));
53-
}
47+
Content = body ?? throw new ArgumentNullException(nameof(body), "object must be set");
48+
Type = type;
5449
}
5550
}
5651
}

src/KubernetesClient/Models/V1Status.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
namespace k8s.Models
44
{
5-
public partial record class V1Status
5+
public partial record V1Status
66
{
77
/// <summary>Converts a <see cref="V1Status"/> object into a short description of the status.</summary>
88
/// <returns>string description of the status</returns>

0 commit comments

Comments
 (0)