redhat-cop / aap_utilities

Ansible Collection for automated deployment of AAP and other objects for general use

https://galaxy.ansible.com/infra/aap_utilities

GNU General Public License v3.0

74 stars 44 forks source link

Skip ansible-lint rules #136

Closed Tompage1994 closed 1 year ago

Tompage1994 commented 1 year ago

What does this PR do?

Fixes lint issues caused by update to ansible-lint 6.13.0

How should this be tested?

CI should pass

Is there a relevant Issue open for this?

N/A

Other Relevant info, PRs, etc

Note that I've set jinja[invalid] to warn for now because the new version has caused a regression in this rule which I've raised a bug for and linked to from the PR in controller_configuration https://github.com/redhat-cop/controller_configuration/pull/503

sean-m-sullivan commented 1 year ago

@Tompage1994 Needs a rebase

branic commented 1 year ago

If there are places where jinja[spacing] can't be fixed then we should turn off the check just for that case and not for the entire project.

Tompage1994 commented 1 year ago

If there are places where jinja[spacing] can't be fixed then we should turn off the check just for that case and not for the entire project.

You're right. I've raised PRs or changed PRs in other repos to fix this. I've updated this PR too