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.

Sign up for GitHub

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

Jump to bottom

Incompatible yamllint configuration should not be blocked when yaml errors are ignored #4152

Closed
felixfontein opened this issue May 12, 2024 · 0 comments · Fixed by #4158
Closed

Incompatible yamllint configuration should not be blocked when yaml errors are ignored #4152

felixfontein opened this issue May 12, 2024 · 0 comments · Fixed by #4158
Assignees
@ssbarnea
Labels
bug

Comments

@felixfontein
Copy link
Contributor

Summary

Reported on IRC in #ansible-community.

I've faced an error since #4139 got merged, it does not respect warn_list: - yaml nor skip_list: - yaml in .ansible-lint and fails.

I think the yamllint configuration check should be ignored in case all yaml errors are ignored in the ansible-lint config file.

Issue Type
  • Bug Report
@felixfontein felixfontein added bug new Triage required labels May 12, 2024
@ssbarnea ssbarnea self-assigned this May 13, 2024
@ssbarnea ssbarnea removed the new Triage required label May 13, 2024
ssbarnea added a commit that referenced this issue May 13, 2024
@ssbarnea
Allow running with incompatible yamllint config
ef8a647 
Fixes: #4152
@ssbarnea ssbarnea mentioned this issue May 13, 2024
Merged
ssbarnea added a commit that referenced this issue May 13, 2024
@ssbarnea
Allow running with incompatible yamllint config
84818c7 
Fixes: #4152
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ssbarnea ssbarnea

Labels
bug
Projects
🧰 devtools project board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow running with incompatible yamllint config ansible/ansible-lint
2 participants
@ssbarnea @felixfontein