Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LSP related improvements for Sublime LSP #513

Merged
merged 5 commits into from
Mar 18, 2022
Merged

LSP related improvements for Sublime LSP #513

merged 5 commits into from
Mar 18, 2022

Conversation

soutaro
Copy link
Owner

@soutaro soutaro commented Mar 17, 2022

Found minor problems when I was testing with Sublime LSP.

  • Pass --log-output command line argument to workers so that we can use tail -f for LSP logs
  • Delete percentage (55%) message from LSP progress since Sublime LSP prints the percentage itself (Why VSCode doesn't print it...)
  • Strip HTML comment from markdown message since Sublime LSP doesn't handle them
  • Filter Goto definition results from workers (VSCode automatically de-dup the list, but Sublime LSP doesn't)
  • Fix RBS type name completion insertion duplication

@soutaro soutaro enabled auto-merge March 18, 2022 00:37
@soutaro soutaro merged commit c4307a6 into master Mar 18, 2022
@soutaro soutaro deleted the better-lsp branch March 18, 2022 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant