On Thu, 2020-05-14 at 11:37 -0700, Kevin Fenzi wrote:
Awesome work!
So nice to have CI (of any kind) running. :)
So what does ansible-review check for currently? whats a failure?
I've looked into it and there isn't an easy answer, except that the way
it's set up right now, we don't gate on it, its output is advisory.
As I understand the code(*), by default ansible-review considers most
things it flags a mere warning, except if a variable is used bare
instead of being flanked by "{{ ... }}". We could make our own
standards and additional lint rules, if we needed, though.
Ciao,
Nils
(*): Yeah, code, call me Luke (no, don't).
--
Nils Philippsen "Those who would give up Essential Liberty to
Software Engineer purchase a little Temporary Safety, deserve neither
Red Hat Liberty nor Safety." -- Benjamin Franklin, 1759
PGP fingerprint: D0C1 1576 CDA6 5B6E BBAE 95B2 7D53 7FCA E9F6 395D
old: C4A8 9474 5C4C ADE3 2B8F 656D 47D8 9B65 6951 3011