Fix SemVer.ToString to handle v0 case (Azure#24101)

azure-sdk · weshaggard · web-flow · commit ec92967fd70f · 2021-09-13T13:03:52.000-07:00
Since we are treating v0 versions as prerelease we need to make sure
we don't accidently start to add the bogus prerelease label in cases where
we call ToString() on the version.

Co-authored-by: Wes Haggard &lt;weshaggard@users.noreply.github.com&gt;
diff --git a/eng/common/scripts/SemVer.ps1 b/eng/common/scripts/SemVer.ps1
@@ -136,7 +136,7 @@ class AzureEngSemanticVersion : IComparable {
   {
     $versionString = "{0}.{1}.{2}" -F $this.Major, $this.Minor, $this.Patch
 
-    if ($this.IsPrerelease)
+    if ($this.IsPrerelease -and $this.PrereleaseLabel -ne "zzz")
     {
       $versionString += $this.PrereleaseLabelSeparator + $this.PrereleaseLabel + `
                         $this.PrereleaseNumberSeparator + $this.PrereleaseNumber

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ class AzureEngSemanticVersion : IComparable {`
`136`	`136`	`{`
`137`	`137`	`$versionString = "{0}.{1}.{2}" -F $this.Major, $this.Minor, $this.Patch`
`138`	`138`
`139`		`- if ($this.IsPrerelease)`
	`139`	`+ if ($this.IsPrerelease -and $this.PrereleaseLabel -ne "zzz")`
`140`	`140`	`{`
`141`	`141`	$versionString += $this.PrereleaseLabelSeparator + $this.PrereleaseLabel + `
`142`	`142`	`$this.PrereleaseNumberSeparator + $this.PrereleaseNumber`