From 8856e0ea8276aca6040bb0da61f41de4cd9da07b Mon Sep 17 00:00:00 2001 From: wucm667 Date: Thu, 7 May 2026 11:48:46 +0800 Subject: [PATCH] docs: remove stale chart version bump guidance from check-contribution skill The check-contribution SKILL.md instructed contributors to bump Chart.yaml versions in feature PRs, but this has been fully delegated to release automation (create-release-pr.yml). Feature PRs should not modify Chart.yaml; version bumps are now bot-driven. Closes #5095 --- .claude/skills/check-contribution/SKILL.md | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/.claude/skills/check-contribution/SKILL.md b/.claude/skills/check-contribution/SKILL.md index b138c8653c..b36b181564 100644 --- a/.claude/skills/check-contribution/SKILL.md +++ b/.claude/skills/check-contribution/SKILL.md @@ -1,6 +1,6 @@ --- name: check-contribution -description: Validates operator chart contribution practices (helm template, ct lint, docs generation, version bump) before committing changes. +description: Validates operator chart contribution practices (helm template, ct lint, docs generation) before committing changes. allowed-tools: [Bash, Read] --- @@ -28,19 +28,14 @@ helm-docs --dry-run ``` Verify that `values.yaml` variables are documented and the generated README.md matches. -### 4. Chart Version Bump -If chart files changed, verify: -- `deploy/charts/operator/Chart.yaml` version is bumped for operator changes -- `deploy/charts/operator-crds/Chart.yaml` version is bumped for CRD changes -- Version follows [SemVer](https://semver.org/) and bump type matches the change scope - ## Output Format ``` ✅ or ❌ Helm template renders successfully ✅ or ❌ Chart linting passes ✅ or ❌ Documentation up-to-date -✅ or ❌ Chart version bumped appropriately + +Note: Chart version bumps are automated by the release bot. Feature PRs should NOT modify Chart.yaml. ``` Include specific errors for any failing checks with actionable remediation commands.