atom / atom-languageclient

Language Server Protocol support for Atom (the basis of Atom-IDE)
https://ide.atom.io/
MIT License
388 stars 78 forks source link

Fix completion deleting the triggerChar #282

Closed alexheretic closed 4 years ago

alexheretic commented 4 years ago

Fixes incorrect completion, e.g. .with -> with_foo() (dot deletion).

Looking at the server messages they seem correct, so this does indeed seem like a client bug. Why was the triggerChar.length being removed? Was this a workaround for some language-server?

This fix is also available in my fork:

"dependencies": {
  "atom-languageclient": "github:alexheretic/atom-languageclient#build",
}