Skip to content
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

Create new tag when version in Chart.yaml is bumped #729

Merged
merged 1 commit into from
Nov 15, 2021

Conversation

jkremser
Copy link
Member

@jkremser jkremser commented Nov 15, 2021

  • The commit message has to contain RELEASE: to trigger this action.
  • The version is taken from the appVersion field from Chart.yaml

tested on my fork: this action was triggered after this pr was merged. And as the result the tag v6.6.6 was created.

Signed-off-by: Jirka Kremser [email protected]

k0da
k0da previously approved these changes Nov 15, 2021
Copy link
Collaborator

@k0da k0da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: [release] -> RELEASE: in commit message

The commit message has to contain 'RELEASE:' to trigger this action.
The version is taken from the appVersion field from Chart.yaml.

Signed-off-by: Jirka Kremser <[email protected]>
Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@jkremser jkremser merged commit e379a66 into k8gb-io:master Nov 15, 2021
@jkremser jkremser deleted the auto-tag branch November 15, 2021 12:27
@jkremser jkremser restored the auto-tag branch November 15, 2021 14:41
@jkremser jkremser deleted the auto-tag branch November 18, 2021 14:35
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.

3 participants