Fix release chart action

zilliztech / milvus-helm

Apache License 2.0

54 stars 41 forks source link

Fix release chart action #88

Closed LoveEachDay closed 4 months ago

LoveEachDay commented 4 months ago

What this PR does / why we need it:

/cc @locustbaby @samhuang-z

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [mychartname])
[ ] PR only contains changes for one chart

yellow-shine commented 4 months ago

/lgtm

sre-ci-robot commented 4 months ago

@samhuang-z: changing LGTM is restricted to collaborators

In response to [this](https://github.com/zilliztech/milvus-helm/pull/88#issuecomment-2097858398): >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

sre-ci-robot commented 4 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LoveEachDay, samhuang-z

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/zilliztech/milvus-helm/blob/master/OWNERS)~~ [LoveEachDay] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

yellow-shine commented 4 months ago

/lgtm