Skip to content

Commit

Permalink
Comment out code due to possible regression
Browse files Browse the repository at this point in the history
  • Loading branch information
gusty authored Feb 2, 2025
1 parent c7fca17 commit 35f2ffe
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions tests/FSharpPlus.Tests/General.fs
Original file line number Diff line number Diff line change
Expand Up @@ -1217,8 +1217,11 @@ module Alternative =
Assert.AreEqual (fullList, SideEffects.get ()) // short-circuits but the conversion to set forces all side-effects

SideEffects.reset ()
let _ = choice (NonEmptyList.ofList (toList t)) // uses Default1 (Choice defined on NonEmptyList)
Assert.AreEqual (fullList, SideEffects.get ()) // short-circuits but the conversion to set forces all side-effects
// Possible F# regression:
// General.fs(1224,25): error FS0465: Type inference problem too complicated (maximum iteration depth reached).
// Consider adding further type annotations. [D:\a\FSharpPlus\FSharpPlus\tests\FSharpPlus.Tests\FSharpPlus.Tests.fsproj]
// let _ = choice (NonEmptyList.ofList (toList t)) // uses Default1 (Choice defined on NonEmptyList)
// Assert.AreEqual (fullList, SideEffects.get ()) // short-circuits but the conversion to set forces all side-effects

SideEffects.reset ()
let _ = choice (WrappedSeqE t) // uses Default2
Expand Down

0 comments on commit 35f2ffe

Please sign in to comment.