Potential fix for pull request finding 'Nested 'if' statements can be combined'

lcawl · github-code-quality[bot] · web-flow · commit cc83135d2f7b · 2025-12-10T16:48:56.000-08:00
Co-authored-by: Copilot Autofix powered by AI &lt;223894421+github-code-quality[bot]@users.noreply.github.com&gt;
diff --git a/src/services/Elastic.Documentation.Services/Changelog/GitHubPrService.cs b/src/services/Elastic.Documentation.Services/Changelog/GitHubPrService.cs
@@ -128,12 +128,10 @@ private static (string? owner, string? repo, int? prNumber) ParsePrUrl(string pr
 		}
 
 		// Handle just a PR number when owner/repo are provided
-		if (int.TryParse(prUrl, out var prNumber))
+		if (int.TryParse(prUrl, out var prNumber) &&
+			!string.IsNullOrWhiteSpace(defaultOwner) && !string.IsNullOrWhiteSpace(defaultRepo))
 		{
-			if (!string.IsNullOrWhiteSpace(defaultOwner) && !string.IsNullOrWhiteSpace(defaultRepo))
-			{
-				return (defaultOwner, defaultRepo, prNumber);
-			}
+			return (defaultOwner, defaultRepo, prNumber);
 		}
 
 		return (null, null, null);

Original file line number	Diff line number	Diff line change
`@@ -128,12 +128,10 @@ private static (string? owner, string? repo, int? prNumber) ParsePrUrl(string pr`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`// Handle just a PR number when owner/repo are provided`
`131`		`- if (int.TryParse(prUrl, out var prNumber))`
	`131`	`+ if (int.TryParse(prUrl, out var prNumber) &&`
	`132`	`+ !string.IsNullOrWhiteSpace(defaultOwner) && !string.IsNullOrWhiteSpace(defaultRepo))`
`132`	`133`	`{`
`133`		`- if (!string.IsNullOrWhiteSpace(defaultOwner) && !string.IsNullOrWhiteSpace(defaultRepo))`
`134`		`- {`
`135`		`- return (defaultOwner, defaultRepo, prNumber);`
`136`		`- }`
	`134`	`+ return (defaultOwner, defaultRepo, prNumber);`
`137`	`135`	`}`
`138`	`136`
`139`	`137`	`return (null, null, null);`