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).