Skip to content

Disable YAML line-length warnings#62

Merged
esentuna merged 1 commit into
mainfrom
codex/disable-yaml-line-length
Jun 1, 2026
Merged

Disable YAML line-length warnings#62
esentuna merged 1 commit into
mainfrom
codex/disable-yaml-line-length

Conversation

@alan-walsh

Copy link
Copy Markdown
Contributor

Summary

  • Disable yamllint line-length so long YAML lines no longer generate warning annotations.
  • Keep the YAML lint workflow and all other yamllint rules unchanged.

Why

  • The 120-character limit was warning-only style guidance, not a YAML, Helm, or Kubernetes validity requirement.
  • These annotations were routinely ignored and added code review noise.

Verification

  • yamllint -f github .yamllint
  • Synthetic 130-character YAML scalar via stdin with yamllint -f github - produced no annotations.

Note: Full-repo yamllint on this Windows checkout reports existing new-lines errors from CRLF worktree line endings, so it was not used as the acceptance signal for this one-rule change.

Copilot AI review requested due to automatic review settings June 1, 2026 18:57

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

This PR updates the repository’s yamllint configuration to stop emitting GitHub annotation warnings for long YAML lines, while keeping the rest of the linting rules and workflow behavior unchanged.

Changes:

  • Disable the line-length rule in .yamllint to prevent line-length warning annotations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@esentuna esentuna merged commit 6d11c48 into main Jun 1, 2026
1 check passed
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