search

atom-community / atom-languageclient

Provide integration support for adding Language Server Protocol servers to Atom.
https://www.npmjs.com/package/atom-languageclient
MIT License
45 stars 13 forks source link

fix: fix removing triggerChar #100

Closed UziTech closed 3 years ago

UziTech commented 4 years ago

fixes #99

I can confirm this fixes replacing the trigger character in multiple autocomplete suggestion packages.

I don't know if this is just a bug from https://github.com/atom/atom-languageclient/pull/264 or if there was a reason triggerPoint was used. Perhaps @Aerijo might be able to give some insight?

I am also having some issues figuring out how to write a test for this. I think I figured it out.