Skip to content

Add pre-commit hook for yaml formatting - #382

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#382
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:32
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 formatting automation via pre-commit and applies consistent formatting across existing YAML configuration/workflow files.

Changes:

  • Add google/yamlfmt hook to .pre-commit-config.yaml.
  • Reformat existing YAML files (remove extra blank lines / normalize indentation and comment spacing).
  • Normalize formatting in GitHub workflow and config YAMLs for consistency.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
nrfTest/nrfcfg.yaml YAML formatting cleanup (whitespace/blank lines) in test NRF config.
nrfTest/nrfcfg_with_custom_webui_url.yaml YAML formatting cleanup (whitespace/blank lines) in alternate test config.
.pre-commit-config.yaml Adds yamlfmt hook and reformats pre-commit config structure.
.golangci.yml Removes extra blank lines to match formatter output.
.github/workflows/stale.yml Removes extra blank lines to match formatter output.
.github/workflows/push.yml Removes extra blank lines to match formatter output.
.github/workflows/main.yml Removes extra blank lines to match formatter output.
.github/dependabot.yml Removes extra blank lines to match formatter output.

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

Comment thread .pre-commit-config.yaml
Comment on lines +35 to +45
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v6.0.0
hooks:
- id: end-of-file-fixer
- id: trailing-whitespace
args:
- --markdown-linebreak-ext=md
- repo: https://github.com/google/yamlfmt
rev: v0.21.0
hooks:
- id: yamlfmt
@gab-arrobo
gab-arrobo merged commit 81d3fcf into omec-project:main Aug 18, 2026
12 checks passed
@gab-arrobo
gab-arrobo deleted the yaml-format branch August 18, 2026 23:34
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