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

Add contributing guide to setup dev environment #470

Merged
merged 5 commits into from
Mar 24, 2024

Conversation

staticf0x
Copy link
Contributor

This was prompted by #216 and I wanted to document how to get started first, because for me at least it was the biggest obstacle (not necessarily because of python-lsp-server itself, but the cooperation with sublime text too). However, I quickly got into troubles as I didn't know what kind of information this guide could and should contain. Not sure if I'll be able to create something good enough without deeper knowledge of pylsp.

@staticf0x
Copy link
Contributor Author

@ccordoba12 Could you advise what info would be important to have in a contributing guide?

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Contributor

@pkulev pkulev left a comment

Choose a reason for hiding this comment

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

I like it, thanks!

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @staticf0x! I left a discussion comment for you below.

CONTRIBUTING.md Outdated

## Configure your editor

In Sublime Text 4, open LSP-pylsp settings and change the path to the `pylsp` command:
Copy link
Member

Choose a reason for hiding this comment

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

I don't know if mentioning one preferred editor over others is a good idea because the setup and development workflow is completely different for each of them. I mean, I could write a guide for Spyder as well (which would be useful for our interested contributors) and it'd be another thing entirely.

So, perhaps having a contributing guide per editor would be better?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand, I initially mentioned ST4 because making it use a custom pylsp path was very difficult, so I felt it was important. But yes, there's too many editors and IDEs to cover that it might not be a good idea to have it in the contributing guide.

So, perhaps having a contributing guide per editor would be better?

That sounds like a good idea.

Copy link
Contributor

Choose a reason for hiding this comment

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

So, perhaps having a contributing guide per editor would be better?

That sounds like a good idea.

Or perhaps none at all? Surely, those who are able and willing to contribute will be able to configure their editor already.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the ST config in favor of a generic point of configuring the editor.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Last suggestions for you @staticf0x, then this should be ready.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@ccordoba12 ccordoba12 added this to the v1.11.0 milestone Mar 24, 2024
@ccordoba12 ccordoba12 changed the title Draft: Add contributing guide to setup dev environment Add contributing guide to setup dev environment Mar 24, 2024
@staticf0x
Copy link
Contributor Author

Done, thanks

Copy link
Member

@ccordoba12 ccordoba12 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 now, thanks @staticf0x!

@ccordoba12 ccordoba12 merged commit 433c6a6 into python-lsp:develop Mar 24, 2024
10 checks passed
@staticf0x staticf0x deleted the contributing-guide branch March 25, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants