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

[Docs] language server pages #1490

Merged
merged 33 commits into from
Dec 3, 2020

Conversation

predragnikolic
Copy link
Member

@predragnikolic predragnikolic commented Nov 24, 2020

added:

  • suggestion for installing LSP-json
  • added instructions for installing plugins like LSP-angular, LSP-graphql and other plugins that were missing.
  • discord icon in the footer
  • ST4 and ST3 Client Configuration tabs

changed:

  • point to LSP-* instantiation instructions instead of manual configuration
  • renamed "Server Configuration" to "Client Configuration"

removed:

  • removed deprecated sourcegraph's go-language server (in favour of gopls)
  • remove polymer editor service
  • remove intellij configuration option (in favour of Eclipse JDT Language Server)
  • remove microsoft's python language server option (in favour of pyright and pyls)
  • removed lua-lsp reference (in favour of lua-ls client configuration)
  • removed SBT, DOTTY scala config references (in favour of LSP-metals)

Should I bring any of the removed configurations back?

I think the best way to see the difference is to start the development server locally, and compare it to the current site https://sublimelsp.github.io/LSP/language_servers/


Note: I noticed that some links are broken (for example the C/C++ guide link). That is due to the /lsp prefix that gh pages adds for repos. I added site_url: https://sublimelsp.github.io/LSP/ to the mkdocs.yaml file, but I am not sure if that is gonna fix it.

Maybe I will have to include the lsp prefix in the links:

- [C/C++ guide](/guides/cplusplus)
+ [C/C++ guide](/lsp/guides/cplusplus)

If you know a solution for the link problem, please let me know :)

As they are not necessary, because the MkDocs theme is configured to generate the hash links automatically
They moved the repo to
https://github.com/Polymer/tools/tree/master/packages/editor-service

but they haven't updated the since on 9 Jul 2019,
they also have a vs code plugin which also havent seen an update.

So I will consider this repo unmaintained and will remove the configuration.
not sure if the setting tcp_port even works.

Feel free to discuss this if you want me to revert this removal.
…vers

When we publish the plugins we will add them to the docs
As the LaTeX plugin is only available on ST 4
for consistency
@jwortmann
Copy link
Member

Looks like all relative links are broken. Could you try

- [list of language servers](/language_servers/)
+ [list of language servers](language_servers.md)

in index.md and

- [C/C++ guide](/guides/cplusplus)
+ [C/C++ guide](guides/cplusplus.md)

https://www.mkdocs.org/user-guide/writing-your-docs/#linking-to-pages

@predragnikolic
Copy link
Member Author

Thanks @jwortmann :)

@predragnikolic predragnikolic marked this pull request as draft November 24, 2020 23:38
@rchl
Copy link
Member

rchl commented Nov 27, 2020

Would it make sense and work for mkdocs.yml to live in the docs folder?

docs/language_servers.md Outdated Show resolved Hide resolved
@predragnikolic
Copy link
Member Author

@rchl It makes sense that docs related files are in the docs folder,
so looked up on how to move the mkdocs file and I stumbled upon this:

mkdocs/mkdocs#1276

I am not that bothered with the mkdocs.yml file being at the root.
But If you are, there is one solution suggested by one of the maintainers of mkdocs:
mkdocs/mkdocs#1276 (comment) (the docs/src solution seems ok)

@rchl
Copy link
Member

rchl commented Nov 28, 2020

@rchl It makes sense that docs related files are in the docs folder,
so looked up on how to move the mkdocs file and I stumbled upon this:

mkdocs/mkdocs#1276

I am not that bothered with the mkdocs.yml file being at the root.
But If you are, there is one solution suggested by one of the maintainers of mkdocs:
mkdocs/mkdocs#1276 (comment) (the docs/src solution seems ok)

From that comment it seems possible if we create /docs/src/ directory and put mkdocs.yml in /docs/. I don't want to make the call though. We'll see what @rwols has to say.

BTW. The /site/ directory should be git-ignored.

EDIT: Sorry for repeating what you've said but I didn't read your comment thoroughly :)

@rwols
Copy link
Member

rwols commented Nov 28, 2020

Putting things in docs/src sounds good to me!

for consistency
docs/site folder is generated when `mkdocs build` command is run
@rwols rwols changed the title Docs/langauge servers page [Docs] language server pages Dec 1, 2020
@predragnikolic predragnikolic marked this pull request as ready for review December 3, 2020 00:22
Copy link
Member

@rwols rwols left a comment

Choose a reason for hiding this comment

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

Looks good to me :)

@predragnikolic predragnikolic merged commit a98e004 into st4000-exploration Dec 3, 2020
@predragnikolic predragnikolic deleted the docs/langauge-servers-page branch December 3, 2020 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants