Skip to content

simple yaml formatter#333

Open
afcollins wants to merge 3 commits intocloud-bulldozer:mainfrom
afcollins:yamlfmt
Open

simple yaml formatter#333
afcollins wants to merge 3 commits intocloud-bulldozer:mainfrom
afcollins:yamlfmt

Conversation

@afcollins
Copy link
Member

Type of change

  • Bug fix
  • Optimization

Description

Adds a Makefile targets for linting and formatting yaml files.

Two manual changes to be formatter compatible:

  1. Quote '{{}}' templates
  2. Add space after comments

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please describe the System Under Test.
  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Add makefile inspired by @rook

Signed-off-by: Andrew Collins <ancollin@redhat.com>
Signed-off-by: Andrew Collins <ancollin@redhat.com>
Signed-off-by: Andrew Collins <ancollin@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Mar 26, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign rsevilla87 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@afcollins afcollins mentioned this pull request Mar 26, 2026
7 tasks
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.

1 participant