trigger characters should contains $ too

[mattn]

I prefer to add $ into triggerChars for variables.

        public CompletionRegistrationOptions GetRegistrationOptions()
        {
            return new CompletionRegistrationOptions
            {
                DocumentSelector = new DocumentSelector(new DocumentFilter { Language = "powershell" }),
                ResolveProvider = true,
                TriggerCharacters = new[] { ".", "-", ":", "\\", "$" }
            };
        }

[SydneyhSmith]

@mattn thanks for opening this issue, we are having trouble reproducing this...what version of editor services are you running?

[mattn]

Possible to reproduce on master branch since $ is not contained in TriggerCharacters.

https://github.com/PowerShell/PowerShellEditorServices/blob/81ba0c0b8efb7ee879f2ede511390ba73b0bc86e/src/PowerShellEditorServices/Services/TextDocument/Handlers/CompletionHandler.cs

[TylerLeonhardt]

I wonder if VS Code is sending over the completion request even though it shouldn't... @mattn which client are you using? Looks like Vim or NeoVim?

[mattn]

I use vim-lsp on vim8.

[SydneyhSmith]

@mattn yeah that looks like a great idea, feel free to open a PR