scalameta / metals-sublime

Sublime Text package for Metals, a language server for Scala
https://packagecontrol.io/packages/LSP-metals
Apache License 2.0
16 stars 10 forks source link

Allow open location to take in the `otherWindow` parameter #81

Closed tgodzik closed 2 years ago

tgodzik commented 2 years ago

Not 100% if this will work, related to the changes in https://github.com/scalameta/metals/pull/3540/files

@ayoub-benali could you test it out after the other PR is merged and released?

ayoub-benali commented 2 years ago

@tgodzik first of all sorry for the delay, I somehow missed this completely :( The change isn't needed because Location is just an alias for a dict, you can see it here: https://github.com/sublimelsp/LSP/blob/main/plugin/core/protocol.py#L186