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: Implement settings resolution snapshot tests #11935

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

snowsignal
Copy link
Member

Summary

Closes #11112.

This PR implements snapshot tests which verify that the resolved configuration, taken from a filesystem configuration value and a set of resolved editor settings, remains consistent. We test this consistency with multiple configuration preferences.

@snowsignal snowsignal added the server Related to the LSP server label Jun 19, 2024
@snowsignal snowsignal added this to the Ruff Server: Stable milestone Jun 19, 2024
@snowsignal snowsignal self-assigned this Jun 19, 2024
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

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.

ruff server: Write snapshot tests for project + editor settings resolution
1 participant