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_list: - yaml[indentation] doesn't work #4344

Open
alexiszamanidis opened this issue Sep 21, 2024 · 2 comments
Open

skip_list: - yaml[indentation] doesn't work #4344

alexiszamanidis opened this issue Sep 21, 2024 · 2 comments
Labels
bug documentation This issue or PR is related to the docs

Comments

@alexiszamanidis
Copy link

Hello,

I want to express my gratitude for the excellent linter you've developed.

I'd like to report an issue regarding the skip_list feature, specifically with:

# .ansible-lint
skip_list:
  - yaml[indentation]

I've added it to my .ansible-lint file, but it doesn't seem to skip these checks as expected. You can review the issue in the following commit where the indentation is enforced after running ansible-lint.

Could you please confirm if I'm missing something or if this might be a bug?

Additionally, I'm curious if there's a way to specify the number of spaces for indentation, as I couldn't find any documentation on this.

Any guidance on this matter would be greatly appreciated.

@alexiszamanidis alexiszamanidis added bug new Triage required labels Sep 21, 2024
@alisonlhart alisonlhart added documentation This issue or PR is related to the docs and removed new Triage required labels Oct 30, 2024
@alisonlhart
Copy link
Contributor

We reviewed the commit and noticed you're using --fix. When using the fix feature, skipping yaml tests generally does not work. We will update our documentation to indicate this, as it currently isn't clear in docs.
Thank you for reporting this!

@alexiszamanidis
Copy link
Author

Hi @alisonlhart

Thank you for the clarification! It’s good to know that YAML tests don’t currently work with "--fix".
Including this in the documentation will definitely help others avoid confusion in the future.

If there’s any workaround or if the feature is planned to be supported in the future, please let me know.
I’d be interested in testing YAML as part of "--fix".

Thanks again for looking into this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug documentation This issue or PR is related to the docs
Projects
Status: Roadmap
Development

No branches or pull requests

2 participants