Automattic / vip-cli

The VIP CLI

https://docs.wpvip.com/vip-cli/

MIT License

58 stars 16 forks source link

Remove unnecessary y/N choice for confirmations #1903

Closed sanmai closed 2 months ago

sanmai commented 2 months ago

Description

Let's fix this double-confirmation:

$ vip config envvar set ABC

✔ Enter the value for ABC: example
✔ Please confirm the input value above (y/N) (y/N)

Pull request checklist

[ ] Update SETUP.md with any new environmental variables.
[ ] Update the documentation.
[ ] Manually test the relevant changes.
[ ] Follow the pull request checklist
[ ] Add/update automated tests as needed.

New release checklist

[ ] Automated tests pass.
[ ] The Preparing for release checklist is completed.

Steps to Test

Outline the steps to test and verify the PR here.

Example:

Check out PR.
Run npm run build
Run ./dist/bin/vip-cookies.js nom
Verify cookies are delicious.

github-actions[bot] commented 2 months ago

Dependency Review

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

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Manifest Files

sonarcloud[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud