atom / atom-languageclient

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

Switch to `protected` `AutoLanguageClient#startExclusiveAdapters(server: ActiveServer)` #189

Closed BoykoAlex closed 6 years ago

BoykoAlex commented 6 years ago

Can we switch AutoLanguageClient#startExclusiveAdapters(server: ActiveServer) to protected? I overrode it in JS code to attach adapters for custom messages outside the LSP protocol... The method is private in TS, can't override it...

damieng commented 6 years ago

We have a specific hook for that called postInitialization that is protected and also has an ActiveServer parameter.

That is how we hook up custom messages in ide-java etc. Does that work for your use case?

BoykoAlex commented 6 years ago

I think it should. I'll try it out. Thanks!-)

damieng commented 6 years ago

Cool. Let me know if it doesn't and I'll see if we can further tweak the hooks :)