Fixup

nblockchain · Feb 2, 2024 · ba3e1b4 · ba3e1b4
1 parent a49b020
commit ba3e1b4
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/Directory.Build.targets b/Directory.Build.targets
@@ -1,10 +1,16 @@
 <!-- Workaround for Central Package Management taken from
      https://github.com/NuGet/Home/issues/11949 and
-     https://github.com/microsoft/MSBuildSdks/blob/22cee25ad037827b54b75b6ee1c59a5bc75a744f/src/CentralPackageVersions/Sdk/Sdk.targets#L70-L76 -->
+     https://github.com/microsoft/MSBuildSdks/blob/22cee25ad037827b54b75b6ee1c59a5bc75a744f/src/CentralPackageVersions/Sdk/Sdk.targets#L70-L76
+
+     TODO: when upgrading to .NET8.0, we can simply use <PackageReference Include="FSharp.Core" VersionOverride="7.0.0" /> for example, and it
+     just works (example of a project that works today this way: fantomas)
+-->
 <Project>
     <ItemGroup Condition=" '$(EnableCentralPackageVersions)' != 'false' ">
-        <PackageReference Update="FSharp.Core"
-                          Condition="'$(MSBuildProjectExtension)' == '.fsproj' And '$(DisableImplicitFSharpCoreReference)' != 'true' And '$(UpdateImplicitFSharpCoreReference)' != 'false'"
-                          IsImplicitlyDefined="true" />
+        <PackageReference
+            Update="FSharp.Core"
+            Condition="'$(MSBuildProjectExtension)' == '.fsproj' And '$(DisableImplicitFSharpCoreReference)' != 'true' And '$(UpdateImplicitFSharpCoreReference)' != 'false'"
+            IsImplicitlyDefined="true"
+        />
     </ItemGroup>
 </Project>