Skip to content

Commit 8fd704e

Browse files
authored
docs: remove stale chart version bump guidance from check-contribution skill (#5211)
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
1 parent 84f606a commit 8fd704e

1 file changed

Lines changed: 3 additions & 8 deletions

File tree

  • .claude/skills/check-contribution

.claude/skills/check-contribution/SKILL.md

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: check-contribution
3-
description: Validates operator chart contribution practices (helm template, ct lint, docs generation, version bump) before committing changes.
3+
description: Validates operator chart contribution practices (helm template, ct lint, docs generation) before committing changes.
44
allowed-tools: [Bash, Read]
55
---
66

@@ -28,19 +28,14 @@ helm-docs --dry-run
2828
```
2929
Verify that `values.yaml` variables are documented and the generated README.md matches.
3030

31-
### 4. Chart Version Bump
32-
If chart files changed, verify:
33-
- `deploy/charts/operator/Chart.yaml` version is bumped for operator changes
34-
- `deploy/charts/operator-crds/Chart.yaml` version is bumped for CRD changes
35-
- Version follows [SemVer](https://semver.org/) and bump type matches the change scope
36-
3731
## Output Format
3832

3933
```
4034
✅ or ❌ Helm template renders successfully
4135
✅ or ❌ Chart linting passes
4236
✅ or ❌ Documentation up-to-date
43-
✅ or ❌ Chart version bumped appropriately
37+
38+
Note: Chart version bumps are automated by the release bot. Feature PRs should NOT modify Chart.yaml.
4439
```
4540

4641
Include specific errors for any failing checks with actionable remediation commands.

0 commit comments

Comments
 (0)