From 8dbfee4436510ee890ecb827e1b63b5e661bc963 Mon Sep 17 00:00:00 2001
From: Gregoire Pailler <gregoire@pailler.fr>
Date: Wed, 7 Nov 2018 10:18:23 +0700
Subject: [PATCH] Properly override versionSuffix for release builds. Add build
 number in CI for release builds

---
 build.cake | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/build.cake b/build.cake
index 04fcd8e..b16ad04 100644
--- a/build.cake
+++ b/build.cake
@@ -43,7 +43,7 @@ Task("Configure")
     // following SevVer 2.0.0 rules. MyGet supports SemVer 2.0.0
     if (isReleaseBuild)
     {
-        versionSuffix = string.Empty;
+        versionSuffix = "\"\"";
     }
     else
     {
@@ -76,8 +76,7 @@ Task("Configure")
     }
 
     var versionPrefix = XmlPeek("./Directory.Build.props", "/Project/PropertyGroup/VersionPrefix");
-    var version = string.Join("-", versionPrefix, versionSuffix);
-
+    var version = isReleaseBuild ? $"{versionPrefix}-release.{buildNumber}" : string.Join("-", versionPrefix, versionSuffix);
     if (AppVeyor.IsRunningOnAppVeyor)
     {
         // Update AppVeyor build version so it will match the build version in assemblies and package
@@ -210,7 +209,7 @@ Task("Publish")
     var apiToken = EnvironmentVariable(config.token);
     if (string.IsNullOrEmpty(apiToken))
     {
-        Error("{0} environment variable not found. Unable to push package on {1}", config.token, config.provider);
+        Warning("{0} environment variable not found. Unable to push package on {1}", config.token, config.provider);
     }
     else
     {