Skip to content

fix: remove obsolete extractVersion#79

Open
yocaba wants to merge 1 commit into
mainfrom
fix/helm-version-update
Open

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

Conversation

@yocaba
Copy link
Copy Markdown
Contributor

@yocaba yocaba commented May 22, 2026

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
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bb3077f3-f08f-4c66-8520-2a9e4ab6d4e4

📥 Commits

Reviewing files that changed from the base of the PR and between e5e5264 and 6e3b47d.

📒 Files selected for processing (1)
  • renovate.json

📝 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.
Description check ✅ Passed The description covers all essential sections: What (closes #78), Why (extractVersion breaks config), Testing (none applied), and a completed checklist with notes.
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.

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

✨ Finishing Touches
🧪 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
Copy link
Copy Markdown
Contributor Author

yocaba commented May 22, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

✅ 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.

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

2 participants