Skip to content

fix: remove obsolete extractVersion#79

Merged
yocaba merged 1 commit into
mainfrom
fix/helm-version-update
May 26, 2026
Merged

fix: remove obsolete extractVersion#79
yocaba merged 1 commit into
mainfrom
fix/helm-version-update

Conversation

@yocaba

@yocaba yocaba commented May 22, 2026

Copy link
Copy Markdown
Contributor

What

Closes #78

Why

extractVersion seems to break the renovate config. And we actually don't need to strip a v in the version tag.

Testing

none

Checklist

  • Tests added/updated n/a
  • No breaking changes (or upgrade path documented above)
  • Readable commit history (squashed and cleaned up as desired)
  • AI code review considered and comments resolved

Summary by CodeRabbit

  • Chores
    • Refined Helm chart dependency management configuration. Updated automated version extraction settings in the dependency management system to streamline how chart versions are identified, tracked, and updated. This adjustment optimizes the process for maintaining Helm-based dependencies.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 22, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This PR fixes a Renovate configuration error by removing an invalid extractVersion setting from the Helm custom manager in renovate.json. The configuration now specifies only datasourceTemplate: "helm" for that manager, resolving the validation issue blocking Renovate operations.

Changes

Renovate Helm Configuration Fix

Layer / File(s) Summary
Remove invalid extractVersion from Helm custom manager
renovate.json
The extractVersion pattern is removed from the Helm custom manager configuration, leaving only datasourceTemplate: "helm" to resolve the Renovate configuration validation error.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • olzemal

Poem

🐰 A config bug fixed with one swift line,
extractVersion gone, Renovate now divine!
From error to grace, the Helm chart flows free,
One tiny deletion, solution we see! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: remove obsolete extractVersion' directly and clearly summarizes the main change made in the pull request.
Linked Issues check ✅ Passed The change directly addresses issue #78 by removing the problematic extractVersion configuration that was causing Renovate errors.
Out of Scope Changes check ✅ Passed All changes are scoped to fixing the Renovate configuration issue in renovate.json as required by issue #78; no unrelated changes present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description follows the required template with all major sections completed: What (with issue reference), Why (clear motivation), Testing (explicitly stated as none), and a filled-out checklist.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/helm-version-update

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yocaba yocaba enabled auto-merge May 22, 2026 12:09
@yocaba

yocaba commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented May 22, 2026

Copy link
Copy Markdown
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@yocaba yocaba merged commit 94886aa into main May 26, 2026
2 checks passed
@yocaba yocaba deleted the fix/helm-version-update branch May 26, 2026 08:17
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.

Action Required: Fix Renovate Configuration

3 participants