Skip to content

Commit 72ab236

Browse files
Use merged_at with closed_at fallback in ParseRawPR
Agent-Logs-Url: https://github.com/NuGet/Entropy/sessions/5c1a69d4-c050-4eb2-a6c8-a64d858343a3 Co-authored-by: Nigusu-Allehu <[email protected]>
1 parent d5be3c0 commit 72ab236

1 file changed

Lines changed: 13 additions & 1 deletion

File tree

NuGetClientPRHealth/GitHubClient.cs

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,13 +173,25 @@ private static async Task ThrowIfErrorAsync(HttpResponseMessage resp, string con
173173
private static RawPR ParseRawPR(JsonElement item)
174174
{
175175
var pr = item.GetProperty("pull_request");
176+
DateTime mergedAt;
177+
if (pr.TryGetProperty("merged_at", out var mergedAtProp) &&
178+
mergedAtProp.ValueKind != JsonValueKind.Null &&
179+
mergedAtProp.TryGetDateTime(out var mergedAtValue))
180+
{
181+
mergedAt = mergedAtValue;
182+
}
183+
else
184+
{
185+
mergedAt = item.GetProperty("closed_at").GetDateTime();
186+
}
187+
176188
return new RawPR(
177189
Number: item.GetProperty("number").GetInt32(),
178190
Title: item.GetProperty("title").GetString()!,
179191
Url: item.GetProperty("html_url").GetString()!,
180192
Author: item.GetProperty("user").GetProperty("login").GetString()!,
181193
CreatedAt: item.GetProperty("created_at").GetDateTime(),
182-
MergedAt: pr.GetProperty("merged_at").GetDateTime());
194+
MergedAt: mergedAt);
183195
}
184196

185197
public void Dispose() => _http.Dispose();

0 commit comments

Comments
 (0)