On Sat, May 16, 2020 at 02:04:57PM +0200, Nils Philippsen wrote:
> 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.I think we may want to at some point. Does it at least catch syntax errors right now? ie, mismatched "s or bad indents, etc? kevin
signature.asc
Description: PGP signature
_______________________________________________ infrastructure mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected]
