Skip to content
This repository was archived by the owner on Oct 4, 2021. It is now read-only.

Commit 2d05953

Browse files
authored
Merge pull request #1974 from mono/fsharp-completion-bug
[F#] Prevents completion window appearing here`let (x|`
2 parents 2c48ec6 + 2ea0460 commit 2d05953

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

main/external/fsharpbinding/MonoDevelop.FSharp.Tests/CompletionTests.fs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ type ``Completion Tests``() =
6969
results |> should contain "completeme"
7070

7171
[<TestCase("let x|")>]
72+
[<TestCase("let (x|")>]
7273
[<TestCase("let! x|")>]
7374
[<TestCase("let in|")>]
7475
[<TestCase("let x |")>]

main/external/fsharpbinding/MonoDevelop.FSharpBinding/FSharpTextEditorCompletion.fs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ module Completion =
187187
None
188188

189189
let (|LetIdentifier|_|) context =
190-
if Regex.IsMatch(context.lineToCaret, "\s?(let!?|override|member|for)\s+[^=(]+$", RegexOptions.Compiled) then
190+
if Regex.IsMatch(context.lineToCaret, "\s?(let!?|override|member|for)\s+[^=:]+$", RegexOptions.Compiled) then
191191
Some LetIdentifier
192192
else
193193
None

0 commit comments

Comments
 (0)