Don't warn compiler API version for F# projects

Turns out that Roslyn doesn't surface the `CompilerApiVersion` property in F# projects. See https://github.com/dotnet/sdk/blob/main/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.PackageDependencyResolution.targets#L214 Fixes devlooped/GitInfo#259
devlooped · Mar 20, 2023 · e6bbe15 · e6bbe15
1 parent 3ac2da4
commit e6bbe15
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/ThisAssembly.Prerequisites/ThisAssembly.Prerequisites.targets b/src/ThisAssembly.Prerequisites/ThisAssembly.Prerequisites.targets
@@ -12,10 +12,12 @@
       <RoslynVersion Condition="'$(RoslynVersion)' == ''">0.0</RoslynVersion>
     </PropertyGroup>
 
-    <!--  -->
+    <!-- Don't enforce compiler version for F#, since it's not surfacing the CompilerApiversion at all. 
+         See https://github.com/dotnet/sdk/blob/main/src/Tasks/Microsoft.NET.Build.Tasks/targets/Microsoft.PackageDependencyResolution.targets#L214 -->
     <Warning Code="THIS001"
              Text="ThisAssembly requires Compiler API (Roslyn) version $(ThisAssemblyMinimumRoslynVersion) or greater."
-             Condition="$([MSBuild]::VersionLessThan('$(RoslynVersion)', '$(ThisAssemblyMinimumRoslynVersion)'))" />
+             Condition="$([MSBuild]::VersionLessThan('$(RoslynVersion)', '$(ThisAssemblyMinimumRoslynVersion)')) and 
+                        ('$(Language)' == 'C#' Or '$(Language)' == 'VB')" />
   </Target>
 
 </Project>