warpnet / salt-lint

A command-line utility that checks for best practices in SaltStack.
https://salt-lint.readthedocs.io/en/latest/
MIT License
154 stars 39 forks source link

Support doubly braced string format #196

Closed BrianSidebotham closed 4 years ago

BrianSidebotham commented 4 years ago

This pull request fixes #192 by allowing certain combinations of doubly curl braces to exist without post or pre brace spaces. This is to allow string formatting strings such as "{{0}}" to exist within Jinja statements.

I think this covers enough of the use-case to be useful. We're linting and not syntax checking so I think this should be suitable.

I have included an increased amount of testing around the brace spaces rule as well.

roaldnefs commented 4 years ago

Thanks @BrianSidebotham for your pull request.

The pylint errors are fixed in #195.