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

Improve development experience #192

Merged
merged 3 commits into from
May 22, 2023
Merged

Improve development experience #192

merged 3 commits into from
May 22, 2023

Conversation

konstin
Copy link
Member

@konstin konstin commented May 16, 2023

Internal improvements that making working on the extension easier:

  • Add standard python logging config, so you can log messages and the show up in the output console of the development host VS Code instance
  • Allow editable install from ruff-lsp into ruff-vscode
  • Extend readme development instructions

README.md Outdated

- Clone the [ruff-lsp](https://github.com/charliermarsh/ruff-lsp) to e.g. `../ruff-lsp`
- Go to `../ruff-lsp` and run `pip install -t ../ruff-vscode/bundled/libs/ -e .`
- If you want to use a custom ruff build, copy or link it to `bundles/libs/bin/ruff`
Copy link
Member

Choose a reason for hiding this comment

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

You can also change the path in the extension settings

Copy link
Member Author

Choose a reason for hiding this comment

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

good point, changed

# Don't repeat every message
"pygls.protocol": {
"level": "WARN",
"handlers": ["stderr"],
Copy link
Member

Choose a reason for hiding this comment

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

Is my assumption correct that this writes to stderr and not the output pane?

Copy link
Member Author

Choose a reason for hiding this comment

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

It also writes to the output pane just in red instead of blue

Copy link
Member

Choose a reason for hiding this comment

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

Oh nice. Is this because the vs code extension redirects stderr to the output panel?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, i think they generally just gather all output

@charliermarsh charliermarsh changed the title Better devevelopment experience Better development experience May 16, 2023
@charliermarsh
Copy link
Member

Will use this opportunity to annoyingly comment on a couple procedural nits:

  1. Would prefer to standardize on imperative titles, like "Improve development experience" over "Better development experience".
  2. In the summary, would be great to include more on the what and why (e.g., "This PR adds instructions to streamline the local development of Ruff LSP alongside the VS Code extension", and so on).

@konstin
Copy link
Member Author

konstin commented May 17, 2023

Will do!

@konstin konstin changed the title Better development experience Improve development experience May 17, 2023
@konstin konstin merged commit 19627f7 into main May 22, 2023
@konstin konstin deleted the dev_experience branch May 22, 2023 12: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