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

ruff server: Introduce the ruff.printDebugInformation command #11827

Closed
wants to merge 9 commits into from

Conversation

snowsignal
Copy link
Contributor

Summary

Closes #11715.

Test Plan

(TODO: I need to open the corresponding VS Code PR before this is testable)

In VS Code, running the Print debug information command should show something like the following in the Output channel:
Screenshot 2024-06-10 at 12 21 07 PM

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.
This reverts commit 4a4197e.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.

Unverified

The committer email address is not verified.
@snowsignal snowsignal added the server Related to the LSP server label Jun 10, 2024
@snowsignal snowsignal changed the title ruff server: Introduce ruff.printDebugInformation command ruff server: Introduce the ruff.printDebugInformation command Jun 10, 2024
Copy link

codspeed-hq bot commented Jun 10, 2024

CodSpeed Performance Report

Merging #11827 will improve performances by 14.2%

Comparing jane/server/command/debug (05e8e5e) with main (521a358)

Summary

⚡ 1 improvements
✅ 29 untouched benchmarks

Benchmarks breakdown

Benchmark main jane/server/command/debug Change
linter/all-with-preview-rules[numpy/globals.py] 917.3 µs 803.2 µs +14.2%

Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@snowsignal
Copy link
Contributor Author

I accidentally pulled in commits from another branch. Closing this in favor of a fresh branch

@snowsignal snowsignal closed this Jun 10, 2024
@snowsignal snowsignal deleted the jane/server/command/debug branch June 10, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
server Related to the LSP server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Server: Debug command
1 participant