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

Remove redundant step from example workflow #204

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

dnskr
Copy link
Contributor

@dnskr dnskr commented Jul 5, 2024

Step Install Helm is not needed in the example workflow and might confuse users.

@dnskr dnskr force-pushed the refine-example-in-readme branch from 53b1057 to c417287 Compare July 5, 2024 15:36
@dnskr
Copy link
Contributor Author

dnskr commented Aug 14, 2024

@davidkarlsen Could you please have a look at the PR?

Copy link

@banjoh banjoh left a comment

Choose a reason for hiding this comment

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

LGTM

@cpanato cpanato closed this Jan 20, 2025
@cpanato cpanato reopened this Jan 20, 2025
@cpanato cpanato closed this Jan 20, 2025
@cpanato cpanato reopened this Jan 20, 2025
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

thanks!

@cpanato cpanato merged commit 134a08b into helm:main Jan 20, 2025
3 checks passed
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