j2lint

j2lint copied to clipboard

Words with hyphens within a string not touching a quote are raising `jinja-variable-format` false positives: For instance: ``` {{ "test with spcial-word in the middle" | indent(2, true) }} ```

jccardonar

`jinja-variable-lower-case` wrongly detects escaped strings as variables

1

## Description The `jinja-variable-lower-case` rule in `j2lint` is incorrectly identifies escaped strings as variables. According to the [Jinja2 documentation](https://jinja.palletsprojects.com/en/3.1.x/templates/#escaping), these strings should be treated as literal strings, not variables. ##...

DeadNews

`jinja-variable-lower-case` wrongly detects printed strings as variables

2

Hi, if printing strings inside of a `{{` statement, the tool complains about `jinja-variable-lower-case`: For example: ``` {{ ' trustGuestRxFilters=\'yes\'' if iftype == 'direct' else '' }} ``` Results in:...

tacerus

cf: https://github.com/aristanetworks/j2lint/issues/53#issuecomment-1404697619

gmuloc

auto-correct option

1

Several linters offer auto-correct options. It would make sense to offer this function given a lot of the style lint is related to easily fixable, but tedious fixes.

ryanmerolle

When possible track line position instead of just line

1

When linting a file, multiple instance of an issue on a line or consolidated into only one error even when said error description is not the same for both errors....

ryanmerolle

Invalid Jinja2 syntax not detected (missing leading {)

2

The following is not detected: ```python3 {% if foo is defined %} bar-option % else %} baz-option {% endif %} ``` The missing leading `{` is not found.

c-po

add Dockerfile

2

Closes #3 A maintainer will need to leverage their standard docker image host publish process & key. This is atleast a start for a docker image.

ryanmerolle

I see this a perfect fit for ci, and thus a docker image would be ideal.

ryanmerolle