Skip to content

Add pre-commit hook for yaml formatting - #555

Merged
gab-arrobo merged 1 commit into
omec-project:mainfrom
gab-arrobo:yaml-format
Aug 18, 2026
Merged

Add pre-commit hook for yaml formatting#555
gab-arrobo merged 1 commit into
omec-project:mainfrom
gab-arrobo:yaml-format

Conversation

@gab-arrobo

Copy link
Copy Markdown
Contributor

No description provided.

@gab-arrobo
gab-arrobo requested review from a team and a lite review from Copilot August 18, 2026 21:52
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds YAML auto-formatting to the repository’s pre-commit toolchain and aligns existing YAML files to the formatter’s output to keep future diffs clean and consistent.

Changes:

  • Added yamlfmt to .pre-commit-config.yaml to format YAML on commit.
  • Reformatted OpenAPI spec YAML (configapi.yaml) to correct/normalize indentation (lists under servers, parameters, and enum).
  • Removed stray blank lines in various YAML configs (GitHub workflows, Dependabot, golangci config) for consistent formatting.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
configapi/api/configapi.yaml Normalizes YAML indentation in the OpenAPI spec (lists/parameters/enums).
.pre-commit-config.yaml Adds google/yamlfmt hook and reformats the file itself.
.golangci.yml Removes an extra blank line to match YAML formatting conventions.
.github/workflows/stale.yml Removes extra blank lines for consistent workflow YAML formatting.
.github/workflows/push.yml Removes extra blank lines for consistent workflow YAML formatting.
.github/workflows/main.yml Removes extra blank lines for consistent workflow YAML formatting.
.github/dependabot.yml Removes extra blank lines for consistent Dependabot YAML formatting.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@gab-arrobo
gab-arrobo merged commit 6647180 into omec-project:main Aug 18, 2026
13 checks passed
@gab-arrobo
gab-arrobo deleted the yaml-format branch August 18, 2026 23:42
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.

3 participants