-
-
Notifications
You must be signed in to change notification settings - Fork 185
ci: move helm release to the release-please workflow #4068
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
✅ Deploy Preview for go-feature-flag-doc-preview canceled.
|
|
/gemini review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request moves the Helm chart versioning into the release-please workflow. The change correctly adds the Chart.yaml to the extra-files to be updated. I have one suggestion to make the configuration more explicit and robust, improving maintainability.
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4068 +/- ##
=======================================
Coverage 83.92% 83.92%
=======================================
Files 137 137
Lines 6813 6813
=======================================
Hits 5718 5718
+ Misses 885 884 -1
- Partials 210 211 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|



Description
move helm release to the release-please workflow