Skip to content

Add TQL language#15394

Open
aljazerzen wants to merge 4 commits intohelix-editor:masterfrom
aljazerzen:lang-tql
Open

Add TQL language#15394
aljazerzen wants to merge 4 commits intohelix-editor:masterfrom
aljazerzen:lang-tql

Conversation

@aljazerzen
Copy link

Adds support for Tenzir Query Language.
The tree sitter grammar: https://github.com/tenzir/tree-sitter-tql

I've copied hightlights and indents queries.
The textobjects are left todo.

Adds support for Tenzir Query Language.
The tree sitter grammar: https://github.com/tenzir/tree-sitter-tql

I've copied hightlights and indents queries.
The textobjects are left todo.
Copy link
Contributor

@m4rch3n1ng m4rch3n1ng left a comment

Choose a reason for hiding this comment

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

a few nitpicks and changes to the queries. helix uses different queries and different conventions than the upstream repo apparently. to see the helix queries, take a look here: https://docs.helix-editor.com/master/themes.html#scopes.

additionally the upstream repo does not have a LICENSE file, which is important to the maintainers. it does mention what it is licensed under in the readme, but i am unsure if that is enough.

aljazerzen and others added 2 commits March 6, 2026 10:03
Co-authored-by: may <63159454+m4rch3n1ng@users.noreply.github.com>
@aljazerzen
Copy link
Author

Thanks for review, I'll ask about license.

@aljazerzen aljazerzen marked this pull request as draft March 6, 2026 09:09
@aljazerzen
Copy link
Author

We've licensed the whole repo under Apache 2.0

@aljazerzen aljazerzen marked this pull request as ready for review March 6, 2026 09:48
@m4rch3n1ng
Copy link
Contributor

the revision in the languages.toml entry still points to the revision before the license change.

also: the readme says the repo is licensed under MIT, while the license you added is apache-2.

@aljazerzen
Copy link
Author

Oh, right. I've updated LICENSE and the languages.toml

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.

2 participants