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

Rewrite / reorganize the troubleshooting documentation #1802

Merged
merged 2 commits into from
Jul 24, 2021

Conversation

rchl
Copy link
Member

@rchl rchl commented Jul 20, 2021

  • common problems moved to the beginning
  • common problems rewritten to be more concise, remove repetition
  • in "Completions not shown after certain keywords" removed note about python completion inhibiting issue as that is fixed
  • other small changes that are hard to summarize

@rchl
Copy link
Member Author

rchl commented Jul 20, 2021

I'll move back stuff to the previous places to hopefully make diff easier to read.
Will revert to current locations after review is done.

@@ -15,13 15,11 @@ If the server is crashing on startup, try running `LSP: Troubleshoot server` fro

## Updating the PATH used by LSP servers

You can confirm that your issue is due to `PATH` being different by starting Sublime Text from the command line so that it inherits your shell's environment.
Sublime Text might see a different `PATH` from what your shell environment uses and might not be able to find the server binary due to that. You can see what ST thinks your `PATH` is by opening the ST console by clicking on *View > Show Console*, and running `import os; os.environ["PATH"]` in that console.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old text started with "You can confirm that your issue is due..." even though there is no any issue in this context.

Comment on lines -64 to -65
* Make sure you have a folder added in your Sublime workspace.
* Make sure the document you are opening lives under that folder.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed as this is no longer the case, we pretty much always start the server. In some cases like LSP-typescript we prevent the start but that gives some feedback in the status bar.


* Scope (eg. `source.php`): Verify this is correct by running "Show Scope Name" from the developer menu.
* Syntax (eg. `Packages\PHP\PHP.sublime-syntax`): Verify by running `view.settings().get("syntax")` in the console.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The syntax obviously no longer matters so removed.

export PATH="/usr/local/bin:$PATH"
```

If, for example, you want to expose a `Node` binary to ST and you have it installed through a version manager like `nvm`, you need to insert its [initialization script](https://github.com/nvm-sh/nvm#install--update-script) in the location specified in [this table](troubleshooting.md#updating-the-path-used-by-lsp-servers)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part about "nvm" moved to the "Updating the PATH used by LSP servers" section

@rwols rwols merged commit 0adac8c into main Jul 24, 2021
@rwols rwols deleted the fix/troubleshooting-docs branch July 24, 2021 11:55
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.

3 participants