Skip to content

Add j2lint linter for Jinja2 templates#5048

Merged
hsanson merged 1 commit intodense-analysis:masterfrom
PsickOSSH:feature/j2lint-linter
Oct 26, 2025
Merged

Add j2lint linter for Jinja2 templates#5048
hsanson merged 1 commit intodense-analysis:masterfrom
PsickOSSH:feature/j2lint-linter

Conversation

@PsickOSSH
Copy link
Contributor

This pull request adds support for j2lint as a linter for Jinja2 templates in ALE.

Fixes #2316

Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Needs some more work before it can be merged:

  • Add help entry to ale-jina.txt file.
  • Add entry to doc/ale-supported-languages-and-tools.txt
  • Add entry to doc/ale.txt
  • Add entry to supported-tools.md
  • Add vader tests

See other PRs as example:

@PsickOSSH
Copy link
Contributor Author

PsickOSSH commented Sep 23, 2025

Thanks for the feedback!

I’ll take care of adding the help entry, updating the documentation files (ale-jinja.txt, ale-supported-languages-and-tools.txt, ale.txt, supported-tools.md), and writing the Vader tests.
I’ll follow the examples from the other PRs you shared and push updates to my branch soon.

@PsickOSSH PsickOSSH force-pushed the feature/j2lint-linter branch from fa3afa3 to 9138b45 Compare October 10, 2025 17:16
@PsickOSSH PsickOSSH force-pushed the feature/j2lint-linter branch from 9138b45 to e0d6a7f Compare October 20, 2025 22:31
@PsickOSSH PsickOSSH requested a review from hsanson October 21, 2025 14:13
@PsickOSSH PsickOSSH force-pushed the feature/j2lint-linter branch from e0d6a7f to 019a34f Compare October 24, 2025 13:51
Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Thanks,

@hsanson hsanson merged commit 9811114 into dense-analysis:master Oct 26, 2025
8 checks passed
Caw3 pushed a commit to Caw3/ale that referenced this pull request Dec 10, 2025
Co-authored-by: Nicolas SCHMAUCH <nic.schmauch@i-0330135t.ac-bordeaux.fr>
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.

Support for j2lint

2 participants