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

Fix action version in README #59

Merged
merged 2 commits into from
Nov 8, 2020
Merged

Fix action version in README #59

merged 2 commits into from
Nov 8, 2020

Conversation

JGiola
Copy link
Contributor

@JGiola JGiola commented Nov 6, 2020

Use current latest released version of the action in the README and not a future unreleased one :)

JGiola and others added 2 commits November 6, 2020 22:38
Use current latest released version of the action in the README and not a future unreleased one

Signed-off-by: Jacopo Andrea Giola <[email protected]>
Signed-off-by: Reinhard Nägele <[email protected]>
Copy link
Member

@unguiculus unguiculus left a comment

Choose a reason for hiding this comment

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

The version has already been bumped in preparation for the release which was just cut. Unfortunately, not quite correct, which I just fixed here.

@unguiculus unguiculus merged commit c3d308d into helm:master Nov 8, 2020
@@ -15,7 +15,7 @@ A GitHub action to turn a GitHub project into a self-hosted Helm chart repo, usi

For more information on inputs, see the [API Documentation](https://developer.github.com/v3/repos/releases/#input)

- `version`: The chart-releaser version to use (default: v1.1.1)
- `version`: The chart-releaser version to use (default: v1.1.0)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don’t get why change this version, the default cart-releaser version used is the v1.1.1, is only the chart-releaser-action version used that was wrong because it wasn’t released already, or I’m missing something?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants