Skip to content

Commit 0d93c92

Browse files
committed
chore: fix more stylecop issues
Signed-off-by: jbrinkman <[email protected]>
1 parent 9d79596 commit 0d93c92

File tree

1 file changed

+13
-10
lines changed

1 file changed

+13
-10
lines changed

src/DotNetApiDiff/ApiExtraction/ApiComparer.cs

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ public IEnumerable<ApiDifference> CompareTypes(IEnumerable<Type> oldTypes, IEnum
206206
}
207207
}
208208

209-
if (foundMatch) break;
209+
if (foundMatch) { break; }
210210
}
211211

212212
// Check for auto-mapping if enabled
@@ -435,7 +435,8 @@ public IEnumerable<ApiDifference> CompareMembers(Type oldType, Type newType)
435435
if (AreSignaturesEquivalent(oldMember.Signature, equivalentNewMember.Signature))
436436
{
437437
// Members are equivalent via type mappings - no difference to report
438-
_logger.LogDebug("Members are equivalent via type mappings: {OldSignature} <-> {NewSignature}",
438+
_logger.LogDebug(
439+
"Members are equivalent via type mappings: {OldSignature} <-> {NewSignature}",
439440
oldMember.Signature, equivalentNewMember.Signature);
440441
continue;
441442
}
@@ -494,7 +495,8 @@ private string ApplyTypeMappingsToSignature(string signature)
494495
var oldTypeNameOnly = mapping.Key.Split('.').Last();
495496
var newTypeNameOnly = mapping.Value.Split('.').Last();
496497

497-
if (oldTypeNameOnly != mapping.Key) // Only if we had a namespace
498+
// Only if we had a namespace
499+
if (oldTypeNameOnly != mapping.Key)
498500
{
499501
mappedSignature = ReplaceTypeNameInSignature(mappedSignature, oldTypeNameOnly, newTypeNameOnly);
500502
}
@@ -515,7 +517,6 @@ private string ReplaceTypeNameInSignature(string signature, string oldTypeName,
515517
// We need to replace type names carefully to avoid partial matches
516518
// For example, when replacing "RedisValue" with "ValkeyValue", we don't want to
517519
// replace "RedisValueWithExpiry" incorrectly
518-
519520
var result = signature;
520521

521522
// Pattern 1: Type name followed by non-word character (space, <, >, ,, etc.)
@@ -549,12 +550,14 @@ private bool AreSignaturesEquivalent(string sourceSignature, string targetSignat
549550
var mappedSourceSignature = ApplyTypeMappingsToSignature(sourceSignature);
550551

551552
return string.Equals(mappedSourceSignature, targetSignature, StringComparison.Ordinal);
552-
} /// <summary>
553-
/// Finds an equivalent member in the target collection based on signature equivalence with type mappings
554-
/// </summary>
555-
/// <param name="sourceMember">The member from the source assembly (could be old or new)</param>
556-
/// <param name="targetMembers">The collection of members from the target assembly (could be new or old)</param>
557-
/// <returns>The equivalent member if found, null otherwise</returns>
553+
}
554+
555+
/// <summary>
556+
/// Finds an equivalent member in the target collection based on signature equivalence with type mappings
557+
/// </summary>
558+
/// <param name="sourceMember">The member from the source assembly (could be old or new)</param>
559+
/// <param name="targetMembers">The collection of members from the target assembly (could be new or old)</param>
560+
/// <returns>The equivalent member if found, null otherwise</returns>
558561
private ApiMember? FindEquivalentMember(ApiMember sourceMember, IEnumerable<ApiMember> targetMembers)
559562
{
560563
// First, try to find a member with the same name - this handles "modified" members

0 commit comments

Comments
 (0)