-
Notifications
You must be signed in to change notification settings - Fork 151
feat: add clearPromptOnDone for some prompts
#395
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
Conversation
|
|
@example/basic • @example/changesets commit: |
|
@43081j, do we need this feature? |
|
sorry i have taken a while to get to this! honestly we probably don't need it yet unless you've seen a use case somewhere. i want to avoid adding functionality unless there is demand for it as thats how alternative libraries ended up bloated if we do one day need this feature, i'd be tempted to just document that you can do it yourself at the call site: const input = await text(opts);
clearTerminal(); // pull this from sister ansi or something |
So, i should close this? |
|
I'll close it for now and we can easily pick it back up one day if needed 👍 The effort is much appreciated still |
In this PR, i add
clearPromptOnDoneoption for some prompts like oninquirer.The prompt who get
clearPromptOnDoneoption are:autocompleteautocompleteMultiselectconfirmgroupMultiselectmultiselectpasswordpathselectKeyselecttext