Skip to content

fix: skip Helm publish when chart unchanged, prevent version collisions#25

Merged
bakayolo merged 1 commit intomainfrom
fix/helm-release-guard
Apr 18, 2026
Merged

fix: skip Helm publish when chart unchanged, prevent version collisions#25
bakayolo merged 1 commit intomainfrom
fix/helm-release-guard

Conversation

@bakayolo
Copy link
Copy Markdown
Collaborator

Problem

  1. Helm chart was published on every tagged release, even when only app code changed
  2. Chart.yaml version could collide with an existing git tag (e.g., v0.3.0 was already tagged when PR feat(chart): support extraContainers, volumes, and volumeMounts #23 set chart version to 0.3.0)

Fix

  • Skip publish: diff charts/ against the previous tag — only publish if chart files changed
  • PR guard: fail CI if Chart.yaml version matches an existing git tag, catching collisions before merge

@bakayolo bakayolo force-pushed the fix/helm-release-guard branch 3 times, most recently from 42fa7b4 to 75911db Compare April 18, 2026 00:01
Skip Helm chart publish if no files under charts/ changed since the
previous tag. Avoids publishing stale charts on app-only releases.

Amp-Thread-ID: https://ampcode.com/threads/T-019d9d9d-a106-729a-a80d-a1c87272834e
Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d9d9d-a106-729a-a80d-a1c87272834e
Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d9d9d-a106-729a-a80d-a1c87272834e
Co-authored-by: Amp <amp@ampcode.com>
@bakayolo bakayolo force-pushed the fix/helm-release-guard branch from 75911db to a1c073a Compare April 18, 2026 00:06
@bakayolo bakayolo merged commit 21e4490 into main Apr 18, 2026
7 checks passed
@bakayolo bakayolo deleted the fix/helm-release-guard branch April 18, 2026 00:10
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.

2 participants