Skip to content

Conversation

@webwarrior-ws
Copy link
Contributor

Workaround for bug in FSharp.Compiler.Services that sometimes results in "FSharp.Compiler.Service cannot yet return this kind of pattern match at ..." exception.

Example code that produces the bug:

module Program =
    type Foo = Foo of string
    do
        typeof<int>.GetCustomAttributes false
        |> ignore

Issue in F# github repo: dotnet/fsharp#8854
There is a merged PR that should address this bug: dotnet/fsharp#16121
But at the moment latest FSharp.Compiler.Services on nuget (43.8.101) doesn't include that fix.

Added regression test for NoPartialFunctions rule for weird
case encountered when linting FSharpLint.Console/Program.fs.
@knocte
Copy link
Collaborator

knocte commented Jan 16, 2024

Issue in F# github repo: dotnet/fsharp#8854
There is a merged PR that should address this bug: dotnet/fsharp#16121
But at the moment latest FSharp.Compiler.Services on nuget (43.8.101) doesn't include that fix.

Why is this info not in the commit msg?

Workaround for bug in FSharp.Compiler.Services that sometimes
results in "FSharp.Compiler.Service cannot yet return this kind of pattern match at ..."
exception (see ``Regression found when parsing Console/Program_fs``
test case).
Issue in F# github repo: [1]. There is a merged PR that should
address this bug: [2]. But at the moment latest
FSharp.Compiler.Services on nuget (43.8.101) doesn't include
that fix.

[1] dotnet/fsharp#8854
[2] dotnet/fsharp#16121
@webwarrior-ws webwarrior-ws force-pushed the nopartialfuncs-regression branch from b7e4340 to 106be3f Compare January 16, 2024 12:29
@webwarrior-ws
Copy link
Contributor Author

Issue in F# github repo: dotnet/fsharp#8854
There is a merged PR that should address this bug: dotnet/fsharp#16121
But at the moment latest FSharp.Compiler.Services on nuget (43.8.101) doesn't include that fix.

Why is this info not in the commit msg?

updated commit message

@knocte knocte merged commit 0d0a308 into fsprojects:master Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants