Skip to content

Commit 4deddb4

Browse files
committed
Small code warning fixes
1 parent fe9e951 commit 4deddb4

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

GitVersionCore/VersionCalculation/BaseVersionCalculators/MergeMessageBaseVersionStrategy.cs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ public override BaseVersion GetVersion(GitVersionContext context)
2828

2929
return baseVersions.Length > 1 ? baseVersions.Aggregate((x, y) => x.SemanticVersion > y.SemanticVersion ? x : y) : baseVersions.SingleOrDefault();
3030
}
31-
public static bool TryParse(Commit mergeCommit, EffectiveConfiguration configuration, out SemanticVersion semanticVersion)
31+
32+
static bool TryParse(Commit mergeCommit, EffectiveConfiguration configuration, out SemanticVersion semanticVersion)
3233
{
3334
string versionPart;
3435
if (Inner(mergeCommit, out versionPart))
@@ -117,7 +118,7 @@ static bool Inner(Commit mergeCommit, out string versionPart)
117118

118119
static bool TryGetPrefix(string target, out string result, string splitter)
119120
{
120-
var indexOf = target.IndexOf(splitter);
121+
var indexOf = target.IndexOf(splitter, StringComparison.Ordinal);
121122
if (indexOf == -1)
122123
{
123124
result = null;
@@ -129,7 +130,7 @@ static bool TryGetPrefix(string target, out string result, string splitter)
129130

130131
static bool TryGetSuffix(string target, out string result, string splitter)
131132
{
132-
var indexOf = target.IndexOf(splitter);
133+
var indexOf = target.IndexOf(splitter, StringComparison.Ordinal);
133134
if (indexOf == -1)
134135
{
135136
result = null;

0 commit comments

Comments
 (0)