-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Ability to select installer when multiple matches are returned. #1620
Comments
How would this work if winget was running non-interactively? |
That's a great question, I was thinking about that, and haven't come up with the "best" solution yet. With the import command, the Id and source are specified. |
Another example for this in an unrelated issue: |
This kind of solution is fine I feel. Are there any plans to integrate it? maybe I can work on it |
@SirusCodes we would love to have a community contributed solution. The first step would be to draft a spec so we can take feedback on the proposed implementation. There is a template and there are a few examples in the specs folder. Please create a draft PR when you're ready for review and feedback 😊 |
I think we would want a feature toggle and some discussion about how an interactive command might work. Some users may also prefer the current experience so they can continue having scripted solutions which don't support interactive behaviors. I'd expect that to be controlled by some kind of setting with a rational default behavior/setting. |
If they have scripted solutions, they would be using IDs and won't ever see this interactive option while using scripts. I don't see a need for a feature toggle 🤔 |
any progress |
#2650 you can see the spec here and further updates will be on this PR I guess |
I ran into the same problem today. I would expect it to be interactive like:
"Multiple packages matched your query. Please select one of the following:
[1] Microsoft.Powershell
[2] Microsoft.PowerShell-Preview"
Anyway thanks for the awesome work 💪
Originally posted by @marvinpuethe in #282 (comment)
The text was updated successfully, but these errors were encountered: