Fix code quality/lint issues: formatting and spelling

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Author: Copilot

src/DemaConsulting.BuildMark/RepoConnectors/GitHubRepoConnector.cs

@@ -156,7 +156,7 @@ public override async Task<List<ItemInfo>> GetChangesBetweenTagsAsync(Version? f
         // Parse PR data and extract changes
         var changes = new List<ItemInfo>();
         var issueNumbers = new HashSet<string>();
-        var prData = new List<(string number, string title, string url, List<string> issueNums)>();
+        var prData = new List<(string number, string title, string url, List<string> issueNumbers)>();
 
         // First pass: collect issue numbers and PR data
         foreach (var prElement in prArray)
@@ -197,11 +197,11 @@ public override async Task<List<ItemInfo>> GetChangesBetweenTagsAsync(Version? f
             {
                 var issuesOutput = await RunCommandAsync("gh", "issue list --state all --limit 1000 --json number,title,url,labels");
                 var issuesDoc = JsonDocument.Parse(issuesOutput);
-                
+
                 foreach (var issueElement in issuesDoc.RootElement.EnumerateArray())
                 {
                     var issueNumber = issueElement.GetProperty("number").GetInt32().ToString();
-                    
+
                     // Only process issues that are referenced by PRs
                     if (!issueNumbers.Contains(issueNumber))
                     {
@@ -270,12 +270,12 @@ public override async Task<List<ItemInfo>> GetChangesBetweenTagsAsync(Version? f
                 // PR has no issues - need to get labels from the pr list data
                 // Find the PR element again to get its labels
                 var prType = "other";
-                var prElement = prArray.FirstOrDefault(pe => 
-                    pe.TryGetProperty("number", out var numProp) && 
+                var prElement = prArray.FirstOrDefault(pe =>
+                    pe.TryGetProperty("number", out var numProp) &&
                     numProp.GetInt32().ToString() == prNumber);
 
-                if (prElement.ValueKind != JsonValueKind.Undefined && 
-                    prElement.TryGetProperty("labels", out var labelsElement) && 
+                if (prElement.ValueKind != JsonValueKind.Undefined &&
+                    prElement.TryGetProperty("labels", out var labelsElement) &&
                     labelsElement.ValueKind == JsonValueKind.Array)
                 {
                     var labels = new List<string>();
@@ -426,7 +426,7 @@ private static string ExtractShasFromCommitsJson(string json)
         {
             var doc = JsonDocument.Parse(json);
             var shas = new List<string>();
-            
+
             foreach (var commit in doc.RootElement.EnumerateArray())
             {
                 if (commit.TryGetProperty("sha", out var shaElement))
@@ -438,7 +438,7 @@ private static string ExtractShasFromCommitsJson(string json)
                     }
                 }
             }
-            
+
             return string.Join('\n', shas);
         }
         catch (JsonException)
@@ -458,7 +458,7 @@ private static string ExtractShasFromCompareJson(string json)
         {
             var doc = JsonDocument.Parse(json);
             var shas = new List<string>();
-            
+
             if (doc.RootElement.TryGetProperty("commits", out var commitsElement))
             {
                 foreach (var commit in commitsElement.EnumerateArray())
@@ -473,7 +473,7 @@ private static string ExtractShasFromCompareJson(string json)
                     }
                 }
             }
-            
+
             return string.Join('\n', shas);
         }
         catch (JsonException)

test/DemaConsulting.BuildMark.Tests/BuildInformationTests.cs

@@ -142,7 +142,7 @@ public async Task BuildInformation_CreateAsync_CollectsIssuesCorrectly()
         Assert.AreEqual("1", buildInfo.Changes[0].Id);
         Assert.AreEqual("Add feature X", buildInfo.Changes[0].Title);
         Assert.AreEqual("https://github.com/example/repo/issues/1", buildInfo.Changes[0].Url);
-        
+
         // Second change should be PR #13 (without issues)
         Assert.AreEqual("#13", buildInfo.Changes[1].Id);
 

test/DemaConsulting.BuildMark.Tests/MockRepoConnectorTests.cs

@@ -133,12 +133,12 @@ public async Task MockRepoConnector_GetChangesBetweenTagsAsync_IncludesPRsWithou
 
         // Assert - Should have both issue #1 (from PR #10) and PR #13 (without issues)
         Assert.HasCount(2, changes);
-        
+
         // First change should be issue #1
         Assert.AreEqual("1", changes[0].Id);
         Assert.AreEqual("Add feature X", changes[0].Title);
         Assert.AreEqual("feature", changes[0].Type);
-        
+
         // Second change should be PR #13
         Assert.AreEqual("#13", changes[1].Id);
         Assert.AreEqual("PR #13", changes[1].Title);