From b962b3f8e6225df2912e04c70be79a0699ef119c Mon Sep 17 00:00:00 2001 From: Lucas Weatherhog <31103312+weatherhog@users.noreply.github.com> Date: Thu, 2 Jul 2026 13:21:02 +0200 Subject: [PATCH] Drop key-ordering enforcement from .yamllint The alphabetical key-ordering rule only existed to keep PR diffs readable. The yaml-diff bot (calling giantswarm/github-workflows yaml-diff.yaml) now posts clean semantic diffs that ignore key reordering, so enforcing key order is no longer necessary and is an unexpected restriction for users. Closes giantswarm/roadmap#4121. Co-Authored-By: Claude Opus 4.8 (1M context) --- .yamllint | 3 +-- CHANGELOG.md | 8 ++++++++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.yamllint b/.yamllint index dc04e8c0..676ab0b4 100644 --- a/.yamllint +++ b/.yamllint @@ -3,7 +3,6 @@ ignore: | *.enc.yaml .github/** .pre-commit-config.yaml -rules: - key-ordering: {} +rules: {} yaml-files: - "*.yaml" diff --git a/CHANGELOG.md b/CHANGELOG.md index bd6254cf..fac85390 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -30,6 +30,14 @@ following [Semantic Versioning](https://semver.org/spec/v2.0.0.html). - migrated `.spec.config` to `.spec.extraConfigs` - Templates: Rename `nginx-ingress-controller` to `ingress-nginx`. ([#85](https://github.com/giantswarm/gitops-template/pull/85)) +### Removed + +- Dropped the alphabetical `key-ordering` rule from `.yamllint`. It only + existed to keep PR diffs readable; the `yaml-diff` bot now provides clean + semantic diffs (ignoring key reordering), so the restriction is no longer + needed. Closes + [giantswarm/roadmap#4121](https://github.com/giantswarm/roadmap/issues/4121). + ## [0.1.0] Initial release - Added