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

Update documentation #427

Merged
merged 15 commits into from
May 30, 2024
Merged

Update documentation #427

merged 15 commits into from
May 30, 2024

Conversation

alcarney
Copy link
Collaborator

@alcarney alcarney commented Dec 22, 2023

As highlighted in #424, our tutorial is a bit out of date and could do with a refresh.

This PR takes the opportunity to update the documentation as a whole, re-organizing it so that it's hopefully easier to navigate. I'm also porting some of the examples that are buried in our issues so that they are easier to find.

I'm using the Diátaxis documentation framework as a guide

TODO

Code review checklist (for code reviewer to complete)

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Commit messages are meaningful (see this for details)
  • Tests have been included and/or updated, as appropriate
  • Docstrings have been included and/or updated, as appropriate
  • Standalone docs have been updated accordingly

@alcarney
Copy link
Collaborator Author

@tombh are you able to enable pr builds for readthedocs?

@tombh
Copy link
Collaborator

tombh commented Dec 22, 2023

I don't yet. I message David about it.

@tombh
Copy link
Collaborator

tombh commented Jan 9, 2024

David said that it should be enabled now.

@alcarney
Copy link
Collaborator Author

Hmm, it does not appear to be working... I'd expect to see a readthedocs item to appear in the list of CI checks and it be linked to a build of the docs generated from this PR 🤔

@tombh
Copy link
Collaborator

tombh commented Jan 15, 2024

Ah 😬 Okay, I've let David know.

@tombh
Copy link
Collaborator

tombh commented Jan 15, 2024

You should have an invite to join the readthedocs.io account.

@alcarney
Copy link
Collaborator Author

I don't see anything wrong in the config... though the docs suggest that resync-ing the webhook might fix it - something that I don't have permission (in GitHub) todo.

@tombh
Copy link
Collaborator

tombh commented Jan 15, 2024

The "Pull requests" toggle wasn't set:
image

So I set it. Does that fix it?

@alcarney
Copy link
Collaborator Author

Does that fix it?

Yes! Thank you 😄

image

You can now click the Details link on the readthedocs check to see a preview of what this PR will do if merged

@noklam
Copy link
Contributor

noklam commented Apr 23, 2024

Would love to see more example of implementations. For example I am looking for how to implement "find reference". I find Jedi's example: https://github.com/pappasam/jedi-language-server/blob/9df0ab5c3481c4e637a8cc1dc3bb5e7afb11c960/jedi_language_server/server.py#L414C1-L428C44

What's the plan or guideline around adding new examples? I may be able to contribute back an example or two later. I think the library is awesome but it's hard to get started with it. Often, I need to look at other libraries source code instead of the docs.

@alcarney
Copy link
Collaborator Author

Have you found our example servers? You can find an example of find references here. Though I admit we don't currently have any prose to explain what is going on.

What's the plan or guideline around adding new examples?

I'd like to convert all of the tests here to end to end tests like these, where each test is backed by a corresponding example server. #440 was the first pass at this and I'm currently working on an example server for the various document formatting methods.

I may be able to contribute back an example or two later.

If you can, that would be awesome :)

I think the library is awesome but it's hard to get started with it

I agree, hopefully, slowly but surely we can turn that around :)

@noklam
Copy link
Contributor

noklam commented May 1, 2024

@alcarney Thanks, I did see the example but somehow missed the find_reference part.

I think one thing that I struggle with is that I don't know how to map the server protocol to something that I can see in an IDE (VSCode). I understand pygls is not only for VSCode. For example what does Go to Reference and Find all reference correspond to? This is not necessary problem of pygls, probably more of VSCode documentation problem. Googling doesn't give me anything immediate useful. I usually have to try the example server, find a way to trigger the code and see it in action to understand what does it actually do.

This example servers is amazing btw, it has the backbone of everything I needed. Thanks again!

If you can, that would be awesome :)

For sure, let me know if there is anything I can help, like reviewing the documentation or testing it.

@noklam
Copy link
Contributor

noklam commented May 2, 2024

https://github.com/microsoft/vscode-extension-samples/tree/main/completions-sample

came across this and I find this is quite helpful.

@alcarney alcarney marked this pull request as ready for review May 29, 2024 20:04
@alcarney alcarney requested a review from tombh May 29, 2024 20:04
@alcarney
Copy link
Collaborator Author

@tombh would you be ok with merging this PR as is?

It's not going to be perfect, but I think this is in a place now where we can start adding to it incrementally

@alcarney alcarney force-pushed the update-docs branch 2 times, most recently from dc94bc3 to 7956754 Compare May 29, 2024 21:36
@tombh
Copy link
Collaborator

tombh commented May 30, 2024

Sure! I had a look at https://pygls--427.org.readthedocs.build/en/427/ and it looks great!

@alcarney
Copy link
Collaborator Author

Thanks!

@alcarney alcarney merged commit bddd3d9 into openlawlibrary:main May 30, 2024
16 checks passed
@alcarney alcarney deleted the update-docs branch May 30, 2024 17:57
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