sublimelsp / LSP-ruff

LSP helper for ruff - an extremely fast Python linter, written in Rust.
https://packagecontrol.io/packages/LSP-ruff
MIT License
31 stars 4 forks source link

fix: ignore REPL views and Python syntax test files #64

Open jfcherng opened 2 months ago

jfcherng commented 2 months ago

Refs


I doubt whether this should be applied generally in upstream LSP or something. Only a very few clients will want to work on those views (e.g., LSP-copilot).

LDAP commented 2 months ago

Disabling in REPL could be done here but disabling in test files should be done in LSP imo. Other opinions?

jfcherng commented 2 months ago

Since SublimeREPL is a 3rd-party package, whether we want to support auto disable it in upstream LSP can be discussed.

But syntax test is a ST built-in, I think it makes sense to disable that by default in upstream LSP indeed.

I've created a related issue in LSP repository: https://github.com/sublimelsp/LSP/issues/2504