-
-
Notifications
You must be signed in to change notification settings - Fork 59
ci(helm-docs): deploy helm chart and update the documentation #2807
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
base: dev
Are you sure you want to change the base?
ci(helm-docs): deploy helm chart and update the documentation #2807
Conversation
Here's the code health analysis summary for commits Analysis Summary
|
c11ffde
to
b4756f3
Compare
Coverage Report
File CoverageNo changed files found. |
b4756f3
to
6f1c366
Compare
6f1c366
to
52fed62
Compare
I've discussed with oben01 to try it out locally this week (With act, that allows to run workflows locally) |
What is the state of this @oben01 ? |
@manuel-rw thank u for the review. I can do the changes. FYI we need @Meierschlumpf to test it with act |
Okay I've finally took the time to try this out and fixed a few small issues on the way. Can you check if it is correct:
|
…-on-release-using-github-action
Because otherwise it seems to tightly coppled to the charts |
Running it locally was pretty simple:
With the following event.json I've used: {
"release": {
"tag_name": "v1.20.1"
}
}
|
LGTM Thank u for your work |
Homarr
Thank you for your contribution. Please ensure that your pull request meets the following pull request:
pnpm build
, autofix withpnpm format:fix
)dev
branchx
,y
,i
or any abbrevation)