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

ci: add linting for yml files #25

Merged
merged 4 commits into from
Sep 11, 2024
Merged

ci: add linting for yml files #25

merged 4 commits into from
Sep 11, 2024

Conversation

dhth
Copy link
Owner

@dhth dhth commented Sep 11, 2024

No description provided.

@dhth dhth merged commit b84edcf into main Sep 11, 2024
7 checks passed
@dhth dhth deleted the add-linting-for-yml-files branch September 11, 2024 16:13
@ccoVeille
Copy link

I'm surprised. Usually yamlfmt is not used for limiting. You will find quite GitHub action for that, it may explain why you had to do the trick to install the latest version

Usually, yamlint is the one used for linting.

Any reason for that?

Originally posted by @ccoVeille in dhth/prs#22 (comment)

You could have used

go install github.com/google/yamlfmt/cmd/yamlfmt@latest

It would have been simpler

Originally posted by @ccoVeille in dhth/prs#22 (comment)

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