Skip to content

Commit

Permalink
Remove the actual reference to FsToolkit.ErrorHandling.TaskResult f…
Browse files Browse the repository at this point in the history
…rom the `TaskSeq.Test` project

We want to remove the dependency in our tests on FsToolkit.ErrorHandling.TaskResult in PR #181 because it uses a high version of FSharp.Core. In other words, it disallows us to properly test TaskSeq with the lowest-denominator FSharp.Core it was compiled with, which we should do, to ensure stability in the widest range of forward-and-backward compatibility. The SmokeTests library, on the other hand, uses the highest RTM release of FSharp.Core and we don't really care about other deps there.
  • Loading branch information
abelbraaksma committed Oct 28, 2023
1 parent e52b8f9 commit 51a16c6
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@
<!-- align test project with minimal required version for TaskSeq -->
<!-- we use 6.0.3 here and not 6.0.2 because TaskResult lib requires it-->
<PackageReference Update="FSharp.Core" Version="6.0.3" />
<PackageReference Include="FsToolkit.ErrorHandling.TaskResult" Version="3.2.0" />
<PackageReference Include="FsUnit.xUnit" Version="5.5.0" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.7.2" />
<PackageReference Include="xunit" Version="2.5.3" />
Expand Down

0 comments on commit 51a16c6

Please sign in to comment.