Merge pull request #669 from dotnet/fix630

AArnott · web-flow · commit 3b3db33b1388 · 2021-10-11T10:40:30.000-06:00
Suppress `cloudBuild.buildNumber: null` being written for `nbgv prepare-release`
diff --git a/src/NerdBank.GitVersioning/VersionOptions.cs b/src/NerdBank.GitVersioning/VersionOptions.cs
@@ -934,6 +934,7 @@ public bool? SetVersionVariables
             /// <summary>
             /// Gets or sets options around how and whether to set the build number preset by the cloud build with one enriched with version information.
             /// </summary>
+            [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
             public CloudBuildNumberOptions? BuildNumber
             {
                 get => this.buildNumber;

Original file line number	Diff line number	Diff line change
`@@ -934,6 +934,7 @@ public bool? SetVersionVariables`
`934`	`934`	`/// <summary>`
`935`	`935`	`/// Gets or sets options around how and whether to set the build number preset by the cloud build with one enriched with version information.`
`936`	`936`	`/// </summary>`
	`937`	`+ [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]`
`937`	`938`	`public CloudBuildNumberOptions? BuildNumber`
`938`	`939`	`{`
`939`	`940`	`get => this.buildNumber;`