Re: Help fixing the ansible-review warnings

[Kevin Sandy <kevin@xxxxxxxxxxxxxxxx>]

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/, but it appears to be empty.

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.

-- kevin

On Nov 21, 2019, at 10:43 AM, Clement Verna <cverna@xxxxxxxxxxxxxxxxx> wrote:

Hi all,

If you would like to help in solving some of the warnings reported by the ansible-review tool, there are a few steps on how to run the tool and get the report in this ticket https://pagure.io/fedora-infrastructure/issue/8157#comment-611372

You can send your patch to this list for review :-)

Thanks

Clément

_______________________________________________
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

_______________________________________________
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