Merge pull request #1 from WorkingRobot/main

Swap breaking change error reporting order for more informative errors

Author: WorkingRobot

BreakingValidators/FieldNamesAndTypes.cs

@@ -16,9 +16,6 @@ public static void Validate(Sheet baseSheet, Sheet newSheet, IReadOnlyList<Excel
 
     private static void ValidateFields(List<Field> baseFields, List<Field> newFields)
     {
-        if (baseFields.Count != newFields.Count)
-            throw new ValidationException("Field count mismatch");
-
         foreach (var field in baseFields)
         {
             var newField = newFields.FirstOrDefault(x => x.Name == field.Name);
@@ -50,6 +47,9 @@ private static void ValidateFields(List<Field> baseFields, List<Field> newFields
                     throw new ValidationException($"Field '{field.Name}' cannot remove targets ({string.Join(", ", targetList)})");
             }
         }
+        
+        if (baseFields.Count != newFields.Count)
+            throw new ValidationException("Field count mismatch");
     }
 
     private static void ValidateRelations(Dictionary<string, List<string>>? baseRelations, Dictionary<string, List<string>>? newRelations)
@@ -69,4 +69,4 @@ private static void ValidateRelations(Dictionary<string, List<string>>? baseRela
                 throw new ValidationException($"Relation '{name}' field list mismatch");
         }
     }
-}
+}