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

Bypass email prompt for upgrades if installed #2332

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

zerbitx
Copy link
Contributor

@zerbitx zerbitx commented Dec 13, 2024

What issue type does this pull request address? (keep at least one, remove the others)
/kind bugfix

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
resolves #ENG-5351

Copy link

netlify bot commented Dec 13, 2024

Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit b875e54
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/675c8700e4038d0008aa266a

Moves prepare call to Run, similar to Destroy
Removes interactive argumment in place of checking itself.
@zerbitx zerbitx requested a review from a team December 16, 2024 21:27
@zerbitx zerbitx marked this pull request as ready for review December 16, 2024 21:27
@cbron cbron merged commit a0f2754 into loft-sh:main Dec 17, 2024
61 checks passed
@loft-bot
Copy link

💚 All backports created successfully

Status Branch Result
v0.22

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

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

Successfully merging this pull request may close these issues.

4 participants