Skip to content

Fix yaml issues in examples #53

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix yaml issues in examples #53

wants to merge 1 commit into from

Conversation

foxish
Copy link
Member

@foxish foxish commented Jul 9, 2025

Ran:

find ./ -name "*.yaml" -o -name "*.yml" | xargs -n 1 yq eval --inplace

This fixes most of the yamllint issues except "line too long (94 > 80 characters)"
It does change the YAML comments not being aligned but that seems ok to me and perhaps more readable for someone looking field by field? Curious to hear thoughts.

@foxish foxish requested a review from scott-cotton July 9, 2025 19:07
@scott-cotton
Copy link
Member

I think it would be great to normalise the indentation of arrays, as that's all over the place.

Too bad for the comments.

Maybe we can re-align the comments and just run this from time to time?

Copy link
Member

@scott-cotton scott-cotton left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants