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

Upgrade helm #50

Merged
merged 2 commits into from
Jan 9, 2021
Merged

Upgrade helm #50

merged 2 commits into from
Jan 9, 2021

Conversation

mrahul17
Copy link
Contributor

No description provided.

Copy link

@carlosstrand carlosstrand left a comment

Choose a reason for hiding this comment

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

Yeah! This change is important since we are facing this issue: #49

@mrahul17
Copy link
Contributor Author

mrahul17 commented Jan 4, 2021

Hi @carlosstrand and @JeroenBoesten thanks for approving this! Can you please verify if it is working for you so that i can finalize the changes?

@JeroenBoesten
Copy link

I can confirm that it solves the issue from #49

@diogopms
Copy link

diogopms commented Jan 6, 2021

Hey team, can we merge this PR? (I need that fix to deploy a new version of my code to production) Thanks 🙏🏻 😢

@mrahul17
Copy link
Contributor Author

mrahul17 commented Jan 6, 2021

@diogopms I am currently using a forked version of this repo with the above changes, given that it is taking time to merge this pr, I would recommend you to do the same

@glend-op
Copy link

glend-op commented Jan 6, 2021

This action is unusable unless this PR is merged.

@colinjfw colinjfw merged commit 0ce1ab7 into deliverybot:master Jan 9, 2021
@mrahul17 mrahul17 deleted the upgrade-helm branch January 12, 2021 07:31
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.

6 participants