Skip to content

Update deployment-operator chart to v0.6.41#765

Merged
michaeljguarino merged 2 commits into
mainfrom
release-v0.6.41
May 23, 2026
Merged

Update deployment-operator chart to v0.6.41#765
michaeljguarino merged 2 commits into
mainfrom
release-v0.6.41

Conversation

@plural-copilot
Copy link
Copy Markdown
Contributor

Update deployment-operator chart to v0.6.41

Plural Service: mgmt/apps

@plural-copilot plural-copilot Bot requested a review from a team as a code owner May 23, 2026 02:58
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 23, 2026

Greptile Summary

Bumps the deployment-operator Helm chart to version v0.6.41, updating both appVersion and version fields and reordering the Chart.yaml keys alphabetically.

  • appVersion is updated from 0.6.40 to v0.6.41 — the v prefix is acceptable here as appVersion is a free-form string.
  • version is updated from 0.6.40 to v0.6.41 — the added v prefix is inconsistent with all prior chart releases which used bare SemVer (0.6.40, 0.6.38, etc.).

Confidence Score: 4/5

A routine chart version bump; the only change worth noting is the newly added v prefix on the version field.

The change is a straightforward version bump. The v prefix introduced on the version field breaks the pattern established by every previous release of this chart, which could cause minor inconsistencies in tooling that performs string comparisons on chart versions, though Helm itself will parse the value correctly.

No files require special attention beyond the version field prefix noted above.

Important Files Changed

Filename Overview
charts/deployment-operator/Chart.yaml Bumps chart and app version to v0.6.41; introduces a v prefix on the version field that is inconsistent with all prior releases of this chart.

Reviews (1): Last reviewed commit: "Update deployment-operator chart to v0.6..." | Re-trigger Greptile

Comment thread charts/deployment-operator/Chart.yaml Outdated
url: https://www.plural.sh
name: deployment-operator
type: application
version: v0.6.41
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.

P2 The version field now has a v prefix (v0.6.41) that wasn't present in any prior release of this chart (e.g. 0.6.40, 0.6.38). While Helm's underlying Masterminds/semver library will accept and strip the prefix, it's inconsistent with the established pattern and the Helm best-practices guide explicitly recommends omitting the v prefix from the version field. The appVersion field is less constrained and the v prefix there is acceptable.

Suggested change
version: v0.6.41
version: 0.6.41

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@michaeljguarino michaeljguarino merged commit 278319c into main May 23, 2026
5 of 6 checks passed
@michaeljguarino michaeljguarino deleted the release-v0.6.41 branch May 23, 2026 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant