Re: Help fixing the ansible-review warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 22, 2019 at 08:09:51AM -0500, Kevin Sandy wrote:
> Are the example lint and standards files included with ansible-review the ones to use, or are there / should there be Fedora Infrastructure specific files? I see there is a repo at https://pagure.io/cpe/ansible-review-templates/ <https://pagure.io/cpe/ansible-review-templates/>, but it appears to be empty.

Yeah, we agreed to use that repo, but then we decided for now at least
that the default rules were fine. 

> One of the warnings for playbooks/check-for-nonvirt-updates.yml is "WARN: Playbook check-for-nonvirt-updates.yml does not present standards version. Using latest standards version 0.1”, and updating the playbook to include the version number from an example file feels a bit odd.

Yeah, that one does. Perhaps we should keep a list of tests that we
don't care for/don't want, and we can supress those?

kevin

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux