Update deployment-operator chart to v0.6.41#765
Conversation
Plural Service: mgmt/apps
Greptile SummaryBumps the
Confidence Score: 4/5A routine chart version bump; the only change worth noting is the newly added The change is a straightforward version bump. The No files require special attention beyond the
|
| 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
| url: https://www.plural.sh | ||
| name: deployment-operator | ||
| type: application | ||
| version: v0.6.41 |
There was a problem hiding this comment.
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.
| 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!
Update deployment-operator chart to v0.6.41
Plural Service: mgmt/apps