-
Notifications
You must be signed in to change notification settings - Fork 74
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
v2.5.0 is giving an error of targetBranch main does not exist
#133
Comments
Happening the same over here. Seems to be related to helm/chart-testing#577 and helm/chart-testing#601 I thought I was getting mad about this output. |
@jessebot i will try to figure out what is going on today, but be kind i do that in my free time |
You are doing a great job @cpanato, thanks for your effort! ;) |
Thanks for your effort @cpanato ! |
Thanks for your hard work, friend! Updated my language, sorry for my tone 💙 |
I suspect that this issue might be caused by helm/chart-testing#511; I'm just running a test to revert the |
I can confirm that setting |
thanks @stevehipwell for being everywhere I need you :)
|
ack, this helm/chart-testing#601 should fix the issue, already tested the main branch here https://github.com/cpanato/testing-ci-providers/actions/runs/6707913084/job/18227518159?pr=1839 and seems to work again running a v3.10.0 release and will update the action soon |
ok this https://github.com/helm/chart-testing/releases/tag/v3.10.0 and the chart-testing-action release https://github.com/helm/chart-testing-action/releases/tag/v2.6.0 should fix this issue a test run here: https://github.com/cpanato/testing-ci-providers/actions/runs/6708194049/job/18228414875?pr=1839 |
updating to fix 2 bug in chart-testing-action helm/chart-testing-action#132 helm/chart-testing-action#133
updating chart-testing-action to fix 2 bugs helm/chart-testing-action#132 helm/chart-testing-action#133
@srinath92 Can you show how you checkout the git repo ? I had the same issue, however I just forgot to set the git checkout with the option |
chart-testing-action: v2.6.1 I can verify that omitting |
you need the |
After #132 was fixed (which it was for me), I seem to be seeing the following error with my job:
Here is my full workflow file
In my own repos, I have moved forward, so this probably won't affect me personally for another week or two, but if I understand correctly, this is also affecting others.
Previous comments pertaining to this issue just for record:
#132 (comment)
#132 (comment)
The text was updated successfully, but these errors were encountered: