Skip to content

fix(dev-env): do not show the update prompt when stdin is not a TTY#2241

Merged
rinatkhaziev merged 1 commit intotrunkfrom
fix/update-prompt
Feb 19, 2025
Merged

fix(dev-env): do not show the update prompt when stdin is not a TTY#2241
rinatkhaziev merged 1 commit intotrunkfrom
fix/update-prompt

Conversation

@sjinks
Copy link
Copy Markdown
Member

@sjinks sjinks commented Feb 19, 2025

Description

When running vip dev-env start < /dev/null, if you happen not to have the latest version of WordPress, VIP CLI will crash on the "Would you like to upgrade WordPress?" prompt because the input is not a TTY.

This PR fixes that.

Pull request checklist

New release checklist

Steps to Test

vip dev-env create -w 6.6 < /dev/null
vip dev-env start < /dev/null

VIP CLI will crash without this patch and start the environment with it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 19, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

@sonarqubecloud
Copy link
Copy Markdown

@rinatkhaziev rinatkhaziev merged commit 0142221 into trunk Feb 19, 2025
@rinatkhaziev rinatkhaziev deleted the fix/update-prompt branch February 19, 2025 20:44
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.

2 participants