Skip to content

feat(check): allow ignoring specific languages#274

Merged
ribru17 merged 1 commit into
masterfrom
check_ignore_langs
Feb 27, 2026
Merged

feat(check): allow ignoring specific languages#274
ribru17 merged 1 commit into
masterfrom
check_ignore_langs

Conversation

@ribru17
Copy link
Copy Markdown
Owner

@ribru17 ribru17 commented Feb 26, 2026

Still TODO: add this as a config option? But maybe it's fine as just a CLI flag, since it is mostly just a QOL change around the directories flag. Plus, someone using the server interactively in an editor will still want diagnostics when opening queries for an ignored language, presumably.

Fixes #271

Still TODO: add this as a config option? But maybe it's fine as just a
CLI flag, since it is mostly just a QOL change around the `directories`
flag. Plus, someone using the server interactively in an editor will
still want diagnostics when opening queries for an ignored language,
presumably.

Fixes #271
@clason
Copy link
Copy Markdown
Contributor

clason commented Feb 26, 2026

Thanks! That would be enough to address my use case, I think. But to clarify:

Plus, someone using the server interactively in an editor will still want diagnostics when opening queries for an ignored language, presumably.

The reason I am interested in such an option is not that I don't want diagnostics for some files, but that I have some files where the serve will error because there is no parser available. And that may happen in editors as well.

@ribru17
Copy link
Copy Markdown
Owner Author

ribru17 commented Feb 27, 2026

Oh, like the ABI is too low such that the LSP actually errors on loading the parser? Does it give an error diagnostic in this case? just curious

@clason
Copy link
Copy Markdown
Contributor

clason commented Feb 27, 2026

No, my use case is if I don't have the parser installed at all (for various reasons including but not limited to obsolete ABI), but I run checks on a queries/ dir that still contains (archived) queries for that parser. (This is admittedly niche.)

@ribru17 ribru17 merged commit 71be64b into master Feb 27, 2026
6 checks passed
@ribru17 ribru17 deleted the check_ignore_langs branch February 27, 2026 09:26
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.

Allow skipping languages for check

2 participants