Skip to content

Commit bb57b58

Browse files
committed
Include time separator in CI prerelease
FAKE's SemVerHelper attempts to parse the prerelease as an Int32, which overflows with the inclusion of the full year. Include a T separator for time to prevent SemVerHelper from trying to be too helpful.
1 parent be6caa6 commit bb57b58

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

build/scripts/Versioning.fsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ module Versioning =
5252
v
5353
| ("canary", Some v) -> failwithf "cannot run canary release, expected no version number to specified but received %s" (v.ToString())
5454
| ("canary", None) ->
55-
let timestampedVersion = (sprintf "ci%s" (DateTime.UtcNow.ToString("yyyyMMddHHmmss")))
55+
let timestampedVersion = (sprintf "ci%s" (DateTime.UtcNow.ToString("yyyyMMddTHHmmss")))
5656
tracefn "Canary suffix %s " timestampedVersion
5757
let canaryVersion = parse ((sprintf "%d.%d.0-%s" currentVersion.Major (currentVersion.Minor + 1) timestampedVersion).Trim())
5858
tracefn "Canary build increased currentVersion (%s) to (%s) " (currentVersion.ToString()) (canaryVersion.ToString())

0 commit comments

Comments
 (0)