Refactor properties to use 'required' instead of nullable with null default

Author: niteshsinghal85

benchmarks/Serialization/SerializationBenchmarkBase.cs

@@ -88,33 +88,33 @@ public sealed class OutgoingResource : Identifiable<long>
         public DayOfWeek Attribute10 { get; set; }
 
         [HasOne]
-        public required OutgoingResource Single1 { get; set; }
+        public OutgoingResource Single1 { get; set; } = null!;
 
         [HasOne]
-        public required OutgoingResource Single2 { get; set; }
+        public OutgoingResource Single2 { get; set; } = null!;
 
         [HasOne]
-        public required OutgoingResource Single3 { get; set; }
+        public OutgoingResource Single3 { get; set; } = null!;
 
         [HasOne]
-        public required OutgoingResource Single4 { get; set; }
+        public OutgoingResource Single4 { get; set; } = null!;
 
         [HasOne]
-        public required OutgoingResource Single5 { get; set; }
+        public OutgoingResource Single5 { get; set; } = null!;
 
         [HasMany]
-        public required ISet<OutgoingResource> Multi1 { get; set; }
+        public ISet<OutgoingResource> Multi1 { get; set; } = null!;
 
         [HasMany]
-        public required ISet<OutgoingResource> Multi2 { get; set; }
+        public ISet<OutgoingResource> Multi2 { get; set; } = null!;
 
         [HasMany]
-        public required ISet<OutgoingResource> Multi3 { get; set; }
+        public ISet<OutgoingResource> Multi3 { get; set; } = null!;
 
         [HasMany]
-        public required ISet<OutgoingResource> Multi4 { get; set; }
+        public ISet<OutgoingResource> Multi4 { get; set; } = null!;
 
         [HasMany]
-        public required ISet<OutgoingResource> Multi5 { get; set; }
+        public ISet<OutgoingResource> Multi5 { get; set; } = null!;
     }
 }

src/Examples/DapperExample/Models/AccountRecovery.cs

@@ -15,5 +15,5 @@ public sealed class AccountRecovery : Identifiable<long>
     public string? EmailAddress { get; set; }
 
     [HasOne]
-    public required LoginAccount Account { get; set; }
+    public LoginAccount Account { get; set; } = null!;
 }

src/Examples/DapperExample/Models/LoginAccount.cs

@@ -14,8 +14,8 @@ public sealed class LoginAccount : Identifiable<long>
     public DateTimeOffset? LastUsedAt { get; set; }
 
     [HasOne]
-    public required AccountRecovery Recovery { get; set; }
+    public AccountRecovery Recovery { get; set; } = null!;
 
     [HasOne]
-    public required Person Person { get; set; }
+    public Person Person { get; set; } = null!;
 }

src/Examples/DapperExample/Models/RgbColor.cs

@@ -19,7 +19,7 @@ public override int? Id
     }
 
     [HasOne]
-    public required Tag Tag { get; set; }
+    public Tag Tag { get; set; } = null!;
 
     [Attr(Capabilities = AttrCapabilities.AllowView)]
     [NotMapped]

src/JsonApiDotNetCore.Annotations/Resources/Annotations/AttrAttribute.cs

@@ -21,7 +21,7 @@ public sealed class AttrAttribute : ResourceFieldAttribute
     /// public class Author : Identifiable<long>
     /// {
     ///     [Attr(Capabilities = AttrCapabilities.AllowFilter | AttrCapabilities.AllowSort)]
-    ///     public string Name { get; set; } = null!;
+    ///     public required string Name { get; set; }
     /// }
     /// ]]></code>
     /// </example>

src/JsonApiDotNetCore.OpenApi.Swashbuckle/JsonApiObjects/AtomicOperations/AddToRelationshipOperation.cs

@@ -12,13 +12,13 @@ internal sealed class AddToRelationshipOperation<TResource> : AtomicOperation
 {
     [Required]
     [JsonPropertyName("op")]
-    public string Op { get; set; } = null!;
+    public required string Op { get; set; }
 
     [Required]
     [JsonPropertyName("ref")]
-    public object Ref { get; set; } = null!;
+    public required object Ref { get; set; }
 
     [Required]
     [JsonPropertyName("data")]
-    public ICollection<IdentifierInRequest<TResource>> Data { get; set; } = null!;
+    public required ICollection<IdentifierInRequest<TResource>> Data { get; set; }
 }

src/JsonApiDotNetCore.OpenApi.Swashbuckle/JsonApiObjects/AtomicOperations/AtomicOperation.cs

@@ -7,5 +7,5 @@ namespace JsonApiDotNetCore.OpenApi.Swashbuckle.JsonApiObjects.AtomicOperations;
 internal abstract class AtomicOperation : IHasMeta
 {
     [JsonPropertyName("meta")]
-    public Meta Meta { get; set; } = null!;
+    public required Meta Meta { get; set; }
 }

src/JsonApiDotNetCore.OpenApi.Swashbuckle/JsonApiObjects/AtomicOperations/AtomicResult.cs

@@ -8,8 +8,8 @@ namespace JsonApiDotNetCore.OpenApi.Swashbuckle.JsonApiObjects.AtomicOperations;
 internal sealed class AtomicResult : IHasMeta
 {
     [JsonPropertyName("data")]
-    public ResourceInResponse Data { get; set; } = null!;
+    public required ResourceInResponse Data { get; set; }
 
     [JsonPropertyName("meta")]
-    public Meta Meta { get; set; } = null!;
+    public required Meta Meta { get; set; }
 }

src/JsonApiDotNetCore.OpenApi.Swashbuckle/JsonApiObjects/AtomicOperations/CreateOperation.cs

@@ -12,9 +12,9 @@ internal sealed class CreateOperation<TResource> : AtomicOperation
 {
     [Required]
     [JsonPropertyName("op")]
-    public string Op { get; set; } = null!;
+    public required string Op { get; set; }
 
     [Required]
     [JsonPropertyName("data")]
-    public DataInCreateRequest<TResource> Data { get; set; } = null!;
+    public required DataInCreateRequest<TResource> Data { get; set; }
 }

src/JsonApiDotNetCore.OpenApi.Swashbuckle/JsonApiObjects/AtomicOperations/DeleteOperation.cs

@@ -12,9 +12,9 @@ internal sealed class DeleteOperation<TResource> : AtomicOperation
 {
     [Required]
     [JsonPropertyName("op")]
-    public string Op { get; set; } = null!;
+    public required string Op { get; set; }
 
     [Required]
     [JsonPropertyName("ref")]
-    public IdentifierInRequest<TResource> Ref { get; set; } = null!;
+    public required IdentifierInRequest<TResource> Ref { get; set; }
 }