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

Jinja language family syntax support #7233

Merged
merged 5 commits into from
Aug 9, 2023
Merged

Jinja language family syntax support #7233

merged 5 commits into from
Aug 9, 2023

Conversation

vanarok
Copy link
Contributor

@vanarok vanarok commented Jun 5, 2023

Add jinja grammar for nunjuncks and jinja languages.
2023-06-05T12:36:51,112551064+06:00

languages.toml Outdated Show resolved Hide resolved
runtime/queries/jinja/highlights.scm Outdated Show resolved Hide resolved
runtime/queries/nunjucks/highlights.scm Outdated Show resolved Hide resolved
languages.toml Outdated Show resolved Hide resolved
the-mikedavis
the-mikedavis previously approved these changes Jun 13, 2023
Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

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

Looks good. Could you resolve the conflicts?

the-mikedavis
the-mikedavis previously approved these changes Jun 19, 2023
Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

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

In the long run I think we'll want a way to detect the language that is being templated. Jinja2 is usually used for HTML but you can use it for any kind of template so usually I see filename.<ext>.j2 where <ext> is html, js, etc. depending on the language being templated. Ideally we would inject that language but that can be left for future work.

@the-mikedavis the-mikedavis added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. A-language-support Area: Support for programming/text languages labels Jun 19, 2023
@the-mikedavis the-mikedavis added S-waiting-on-review Status: Awaiting review from a maintainer. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 9, 2023
@pascalkuthe pascalkuthe merged commit 57f093d into helix-editor:master Aug 9, 2023
6 checks passed
@dlq84
Copy link

dlq84 commented Aug 9, 2023

Thank you so much for this. Now the only language I'm missing is pugjs :)

dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
* feat: add jinja language support

* feat: add nunjucks language support

* feat: add to lang support book jinja and nunjucks languages
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
* feat: add jinja language support

* feat: add nunjucks language support

* feat: add to lang support book jinja and nunjucks languages
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
* feat: add jinja language support

* feat: add nunjucks language support

* feat: add to lang support book jinja and nunjucks languages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-language-support Area: Support for programming/text languages S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants