Skip to content

Commit 1169f38

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. (cherry picked from commit bb57b58)
1 parent 0ed05c3 commit 1169f38

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

build/scripts/Versioning.fsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
#r @"FakeLib.dll"
44
#r @"FSharp.Data.dll"
55
#r @"System.Xml.Linq.dll"
6+
#nowarn "0044" //TODO sort out FAKE 5
67

78
#load @"Projects.fsx"
89
#load @"Paths.fsx"
@@ -51,7 +52,7 @@ module Versioning =
5152
v
5253
| ("canary", Some v) -> failwithf "cannot run canary release, expected no version number to specified but received %s" (v.ToString())
5354
| ("canary", None) ->
54-
let timestampedVersion = (sprintf "ci%s" (DateTime.UtcNow.ToString("MMddHHmmss")))
55+
let timestampedVersion = (sprintf "ci%s" (DateTime.UtcNow.ToString("yyyyMMddTHHmmss")))
5556
tracefn "Canary suffix %s " timestampedVersion
5657
let canaryVersion = parse ((sprintf "%d.%d.0-%s" currentVersion.Major (currentVersion.Minor + 1) timestampedVersion).Trim())
5758
tracefn "Canary build increased currentVersion (%s) to (%s) " (currentVersion.ToString()) (canaryVersion.ToString())

0 commit comments

Comments
 (0)