Skip to content

Commit 04ee73f

Browse files
committed
Final tweaks
1 parent 9555bf9 commit 04ee73f

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

build.fsx

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -461,9 +461,7 @@ let getReleaseNotes currentRelease (lastRelease: GithubRelease) =
461461
let queryResult =
462462
Cli
463463
.Wrap("gh")
464-
.WithArguments(
465-
$"pr list -S \"state:closed base:main closed:>{date} -author:app/robot\" --json commits,mergedAt"
466-
)
464+
.WithArguments($"pr list -S \"state:closed base:main closed:>{date}\" --json commits,mergedAt")
467465
.WithValidation(CommandResultValidation.None)
468466
.ExecuteBufferedAsync()
469467
.Task.Result
@@ -490,15 +488,12 @@ let getReleaseNotes currentRelease (lastRelease: GithubRelease) =
490488
let mergedAtOpt =
491489
match pr.TryGetProperty("mergedAt") with
492490
| Some mergedAtJson ->
493-
try
494-
let mergedAtStr = mergedAtJson.AsString()
495-
Some(
496-
DateTime
497-
.Parse(mergedAtStr, null, System.Globalization.DateTimeStyles.RoundtripKind)
498-
.ToUniversalTime()
499-
)
500-
with _ ->
501-
None
491+
let mergedAtStr = mergedAtJson.AsString()
492+
match
493+
DateTime.TryParse(mergedAtStr, null, System.Globalization.DateTimeStyles.RoundtripKind)
494+
with
495+
| true, dt -> Some(dt.ToUniversalTime())
496+
| false, _ -> None
502497
| None -> None
503498

504499
match mergedAtOpt with

0 commit comments

Comments
 (0)