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

docs: only run chart install if changed #105

Closed
wants to merge 1 commit into from

Conversation

fty4
Copy link
Contributor

@fty4 fty4 commented Jan 6, 2023

If the chart did not change no kind cluster will be created - also the ct install can not be triggered.

Signed-off-by: Marco Lecheler marco@task.media

If the chart did not change no kind cluster will be created - also the ct install can not be triggered

Signed-off-by: Marco Lecheler <marco@task.media>
fty4 added a commit to fty4/chart-testing-action that referenced this pull request Jan 6, 2023
If no change on the chart was detected the linting should not be executed

related to helm#105

Signed-off-by: Marco Lecheler <marco@task.media>
cpanato pushed a commit that referenced this pull request Mar 24, 2023
If no change on the chart was detected the linting should not be executed

related to #105

Signed-off-by: Marco Lecheler <marco@task.media>
@cpanato
Copy link
Member

cpanato commented Mar 24, 2023

need rebase

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.

need rebase

@fty4
Copy link
Contributor Author

fty4 commented Mar 24, 2023

no rebase required - already implemented with #115

@fty4 fty4 closed this Mar 24, 2023
@fty4 fty4 deleted the docs/wf-example-if-ctinstall branch March 24, 2023 11:23
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.

2 participants