Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip ansible-lint rules #227

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Conversation

Tompage1994
Copy link
Contributor

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 redhat-cop/infra.aap_configuration#503

Copy link
Contributor

@djdanielsson djdanielsson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@branic branic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Tompage1994 Tompage1994 mentioned this pull request Feb 21, 2023
@Tompage1994
Copy link
Contributor Author

CI testing will fail until #230 fixed

@Tompage1994 Tompage1994 merged commit 530e6a4 into ansible:devel Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants