Fixed broken build after reformat

src/JsonApiDotNetCore/Hooks/Internal/Discovery/HooksDiscovery.cs

@@ -67,7 +67,7 @@ private void DiscoverImplementedHooks(Type containerType)
 
  foreach (ResourceHook hook in _allHooks)
  {
- MethodInfo? method = containerType.GetMethod(hook.ToString("G"));
+ MethodInfo method = containerType.GetMethod(hook.ToString("G"));
 
  if (method == null || method.DeclaringType == _boundResourceDefinitionType)
  {

src/JsonApiDotNetCore/Hooks/Internal/Execution/HookExecutorHelper.cs

@@ -280,7 +280,7 @@ private void AddToImplicitlyAffected(IEnumerable includedLefts, RelationshipAttr
 
  private static void AddToList(IList list, IEnumerable itemsToAdd)
  {
- foreach (object? item in itemsToAdd)
+ foreach (object item in itemsToAdd)
  {
  list.Add(item);
  }

src/JsonApiDotNetCore/Hooks/Internal/ResourceHookExecutor.cs

@@ -448,7 +448,7 @@ private void ValidateHookResponse<T>(IEnumerable<T> returnedList, ResourcePipeli
  /// </summary>
  private IEnumerable CallHook(IResourceHookContainer container, ResourceHook hook, object[] arguments)
  {
- MethodInfo? method = container.GetType().GetMethod(hook.ToString("G"));
+ MethodInfo method = container.GetType().GetMethod(hook.ToString("G"));
  // note that some of the hooks return "void". When these hooks, the 
  // are called reflectively with Invoke like here, the return value
  // is just null, so we don't have to worry about casting issues here.

src/JsonApiDotNetCore/Hooks/Internal/Traversal/RelationshipProxy.cs

@@ -67,7 +67,7 @@ public object GetValue(IIdentifiable resource)
  return null;
  }
 
- foreach (object? throughResource in throughResources)
+ foreach (object throughResource in throughResources)
  {
  var rightResource = (IIdentifiable)hasManyThrough.RightProperty.GetValue(throughResource);
 
@@ -110,7 +110,7 @@ public void SetValue(IIdentifiable resource, object value)
  var filteredList = new List<object>();
  IList rightResources = TypeHelper.CopyToList((IEnumerable)value, RightType);
 
- foreach (object? throughResource in throughResources ?? Array.Empty<object>())
+ foreach (object throughResource in throughResources ?? Array.Empty<object>())
  {
  if (rightResources.Contains(hasManyThrough.RightProperty.GetValue(throughResource)))
  {

src/JsonApiDotNetCore/Hooks/Internal/Traversal/TraversalHelper.cs

@@ -147,7 +147,7 @@ private Dictionary<RightType, List<KeyValuePair<RelationshipProxy, List<IIdentif
  ExtractLeftResources(leftResources, relationships, rightResourcesGrouped, leftResourcesGrouped);
  }
 
- MethodInfo? processResourcesMethod = GetType().GetMethod(nameof(ProcessResources), BindingFlags.NonPublic | BindingFlags.Instance);
+ MethodInfo processResourcesMethod = GetType().GetMethod(nameof(ProcessResources), BindingFlags.NonPublic | BindingFlags.Instance);
 
  foreach (KeyValuePair<RelationshipProxy, List<IIdentifiable>> kvp in rightResourcesGrouped)
  {

src/JsonApiDotNetCore/Middleware/JsonApiRoutingConvention.cs

@@ -153,7 +153,7 @@ private Type ExtractResourceTypeFromController(Type type)
 
  while (!currentType.IsGenericType || currentType.GetGenericTypeDefinition() != baseControllerType)
  {
- Type? nextBaseType = currentType.BaseType;
+ Type nextBaseType = currentType.BaseType;
 
  if ((nextBaseType == aspNetControllerType || nextBaseType == coreControllerType) && currentType.IsGenericType)
  {

src/JsonApiDotNetCore/Middleware/TraceLogWriter.cs

@@ -76,7 +76,7 @@ private static void WriteProperty(StringBuilder builder, PropertyInfo property,
  builder.Append(property.Name);
  builder.Append(": ");
 
- object? value = property.GetValue(instance);
+ object value = property.GetValue(instance);
 
  if (value == null)
  {
@@ -111,7 +111,7 @@ private static bool HasToStringOverload(Type type)
  {
  if (type != null)
  {
- MethodInfo? toStringMethod = type.GetMethod("ToString", Array.Empty<Type>());
+ MethodInfo toStringMethod = type.GetMethod("ToString", Array.Empty<Type>());
 
  if (toStringMethod != null && toStringMethod.DeclaringType != typeof(object))
  {

src/JsonApiDotNetCore/Resources/Annotations/HasManyThroughAttribute.cs

@@ -132,7 +132,7 @@ public override object GetValue(object resource)
  {
  ArgumentGuard.NotNull(resource, nameof(resource));
 
- object? throughEntity = ThroughProperty.GetValue(resource);
+ object throughEntity = ThroughProperty.GetValue(resource);
 
  if (throughEntity == null)
  {

src/JsonApiDotNetCore/Serialization/JsonApiReader.cs

@@ -221,7 +221,7 @@ private bool HasMissingId(object model)
  /// </summary>
  private bool HasMissingId(IEnumerable models)
  {
- foreach (object? model in models)
+ foreach (object model in models)
  {
  if (TryGetId(model, out string id) && id == null)
  {

src/JsonApiDotNetCore/TypeHelper.cs

@@ -294,7 +294,7 @@ public static bool TypeCanContainHashSet(Type collectionType)
  /// </summary>
  public static Type GetIdType(Type resourceType)
  {
- PropertyInfo? property = resourceType.GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = resourceType.GetProperty(nameof(Identifiable.Id));
 
  if (property == null)
  {
@@ -375,7 +375,7 @@ public static IEnumerable CopyToTypedCollection(IEnumerable source, Type collect
  Type concreteCollectionType = ToConcreteCollectionType(collectionType);
  dynamic concreteCollectionInstance = CreateInstance(concreteCollectionType);
 
- foreach (object? item in source)
+ foreach (object item in source)
  {
  concreteCollectionInstance.Add((dynamic)item);
  }

test/JsonApiDotNetCoreExampleTests/IntegrationTests/QueryStrings/Filtering/FilterDataTypeTests.cs

@@ -44,7 +44,7 @@ public async Task Can_filter_equality_on_type(string propertyName, object value)
  {
  // Arrange
  var resource = new FilterableResource();
- PropertyInfo? property = typeof(FilterableResource).GetProperty(propertyName);
+ PropertyInfo property = typeof(FilterableResource).GetProperty(propertyName);
  property?.SetValue(resource, value);
 
  await _testContext.RunOnDatabaseAsync(async dbContext =>
@@ -256,7 +256,7 @@ public async Task Can_filter_is_null_on_type(string propertyName)
  {
  // Arrange
  var resource = new FilterableResource();
- PropertyInfo? property = typeof(FilterableResource).GetProperty(propertyName);
+ PropertyInfo property = typeof(FilterableResource).GetProperty(propertyName);
  property?.SetValue(resource, null);
 
  var otherResource = new FilterableResource

test/JsonApiDotNetCoreExampleTests/IntegrationTests/ReadWrite/Creating/CreateResourceTests.cs

@@ -110,7 +110,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  workItemInDatabase.DueAt.Should().Be(newWorkItem.DueAt);
  });
 
- PropertyInfo? property = typeof(WorkItem).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(WorkItem).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(int));
  }
 
@@ -157,7 +157,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  userAccountInDatabase.LastName.Should().Be(newUserAccount.LastName);
  });
 
- PropertyInfo? property = typeof(UserAccount).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(UserAccount).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(long));
  }
 
@@ -201,7 +201,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  groupInDatabase.Name.Should().Be(newGroup.Name);
  });
 
- PropertyInfo? property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(Guid));
  }
 

test/JsonApiDotNetCoreExampleTests/IntegrationTests/ReadWrite/Creating/CreateResourceWithClientGeneratedIdTests.cs

@@ -69,7 +69,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  groupInDatabase.Name.Should().Be(newGroup.Name);
  });
 
- PropertyInfo? property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(Guid));
  }
 
@@ -115,7 +115,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  groupInDatabase.Name.Should().Be(newGroup.Name);
  });
 
- PropertyInfo? property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(Guid));
  }
 
@@ -155,7 +155,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  colorInDatabase.DisplayName.Should().Be(newColor.DisplayName);
  });
 
- PropertyInfo? property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(string));
  }
 
@@ -195,7 +195,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  colorInDatabase.DisplayName.Should().Be(newColor.DisplayName);
  });
 
- PropertyInfo? property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(string));
  }
 

test/JsonApiDotNetCoreExampleTests/IntegrationTests/ReadWrite/Updating/Resources/UpdateResourceTests.cs

@@ -209,7 +209,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  groupInDatabase.IsPublic.Should().Be(existingGroup.IsPublic);
  });
 
- PropertyInfo? property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(WorkItemGroup).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(Guid));
  }
 
@@ -256,7 +256,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
  colorInDatabase.DisplayName.Should().Be(newDisplayName);
  });
 
- PropertyInfo? property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
+ PropertyInfo property = typeof(RgbColor).GetProperty(nameof(Identifiable.Id));
  property.Should().NotBeNull().And.Subject.PropertyType.Should().Be(typeof(string));
  }
 

test/TestBuildingBlocks/FakerContainer.cs

@@ -22,7 +22,7 @@ private static MethodBase GetTestMethod()
  {
  var stackTrace = new StackTrace();
 
- MethodBase? testMethod = stackTrace.GetFrames().Select(stackFrame => stackFrame?.GetMethod()).FirstOrDefault(IsTestMethod);
+ MethodBase testMethod = stackTrace.GetFrames().Select(stackFrame => stackFrame?.GetMethod()).FirstOrDefault(IsTestMethod);
 
  if (testMethod == null)
  {

test/UnitTests/Serialization/Common/BaseDocumentParserTests.cs

@@ -142,7 +142,7 @@ public void DeserializeAttributes_VariousDataTypes_CanDeserialize(string member,
 
  // Assert
  PropertyInfo pi = ResourceGraph.GetResourceContext("testResource").Attributes.Single(attr => attr.PublicName == member).Property;
- object? deserializedValue = pi.GetValue(resource);
+ object deserializedValue = pi.GetValue(resource);
 
  if (member == "intField")
  {