search

saltstack-formulas / postfix-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
26 stars 128 forks source link

[salt-lint] Switch tabs to spaces in `postfix/files/mapping.j2` #101

Open myii opened 5 years ago

myii commented 5 years ago

Introducing salt-lint, got a violation triggered here:

https://github.com/saltstack-formulas/postfix-formula/blob/561d0940291a551bc01ce38594773f6e9eee242b/postfix/files/mapping.j2#L11-L13

Examining postfix/files/mapping.j2 of type state
[203] Most files should not contain tabs
postfix/files/mapping.j2:11
{{ key }}{% if colon %}:{% endif %}     {{ value|join(", ") }}

[203] Most files should not contain tabs
postfix/files/mapping.j2:13
{{ key }}{% if colon %}:{% endif %}     {{ value }}

@0xf10e @alxwr Are the tabs absolutely necessary here or can they be changed to spaces?

0xf10e commented 5 years ago

I don't think tabs are necessary here.

myii commented 5 years ago

Thanks for the feedback, @0xf10e.