Merge pull request #1174 from Numpsy/no_fsharp_nowarn

Don't try to disable CA2243 warnings in the generated version info files for F#

Author: AArnott

src/Nerdbank.GitVersioning.Tasks/AssemblyVersionInfo.cs

@@ -709,9 +709,14 @@ public FSharpCodeGenerator(string ns)
             {
             }
 
+            protected override IEnumerable<string> WarningCodesToSuppress { get; } = [];
+
             internal override void AddAnalysisSuppressions()
             {
-                this.CodeBuilder.AppendLine($"#nowarn {string.Join(" ", this.WarningCodesToSuppress.Select(c => $"\"{c}\""))}");
+                if (this.WarningCodesToSuppress.Any())
+                {
+                    this.CodeBuilder.AppendLine($"#nowarn {string.Join(" ", this.WarningCodesToSuppress.Select(c => $"\"{c}\""))}");
+                }
             }
 
             internal override void AddComment(string comment)

test/Nerdbank.GitVersioning.Tests/AssemblyInfoTest.cs

@@ -61,7 +61,6 @@ public void FSharpGenerator(bool? thisAssemblyClass)
 // </auto-generated>
 //------------------------------------------------------------------------------
 
-#nowarn ""CA2243""
 
 namespace AssemblyInfo
 [<assembly: global.System.Reflection.AssemblyVersionAttribute(""1.3.0.0"")>]
@@ -121,7 +120,6 @@ public void FSharpGeneratorWithNamespace(string thisAssemblyNamespace, string ro
 // </auto-generated>
 //------------------------------------------------------------------------------
 
-#nowarn ""CA2243""
 
 namespace {(
     !string.IsNullOrWhiteSpace(thisAssemblyNamespace)