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

[interactive] show all update prior to ask for all #72

Closed
sylvain-bougnoux opened this issue Jan 24, 2019 · 8 comments
Closed

[interactive] show all update prior to ask for all #72

sylvain-bougnoux opened this issue Jan 24, 2019 · 8 comments

Comments

@sylvain-bougnoux
Copy link

pip-review -i shows the 1st update and ask for yes, no, all, quit. It would be convenient to show all possible updates prior to pressing 'A', because sometimes some packages must not be updated. If yes or no, only the 1st one will be treated. I understand I can call pip list -o before, but it is a bit long with many packages.

My 2c.

Thanks for this very useful tool anyway.

@jgonggrijp
Copy link
Owner

Thank you for taking the effort to give feedback.

I'm not sure I understand the issue. You would like to see all the possible updates, but you find pip list -o a bit long. Surely, you expect to see the same list of possible updates from pip-review? In fact, this is almost exactly what you get to see when you run pip-review (without the -i or -a flag).

@sylvain-bougnoux
Copy link
Author

Yes I expect same list. Of course I can do pip list -o or pip-review, but then I have to wait for the answer. Then once validated (from me), recall pip-review -i, and wait a second time, then press 'A'. Overall I have to wait twice. While we can have these possible workflows:

  • either pip-review -i, all possible updates are shown, then press y,n,a, or q; but it is a bit ambiguous, e.g. on the impact of 'y'.
  • either 'pip-review -i', show only the first possible update (current solution) but add 'S' key for show, that will display all possible updates, then press 'A' as usual.

With these 2 workflows we have to wait just once. Up to you.

@jgonggrijp
Copy link
Owner

Thanks for the explanation. Pull requests and more opinions welcome.

@jgonggrijp
Copy link
Owner

@sylvain-bougnoux I suddenly wondered. Doesn't pip-review followed by pip-review -aapproximately do what you describe?

@sylvain-bougnoux
Copy link
Author

Actually yes. The only issue is that you have to wait twice for pip-review to know what to do. One pip-review call takes for me about 20-30s to respond. It is awkward to have to wait twice.
Thanks

@jgonggrijp
Copy link
Owner

Ah, you mentioned the having to wait twice before. Sorry for overlooking that in my previous comment. I find the use case very reasonable.

How about the following. Add a --batch option which, when used together with --interactive (together -ib in short), will cause all available updates to show immediately, followed by a single question: install all updates? (y/n).

Please don't wait for me to implement it, though. See #76. I'll welcome a pull request that implements something like this.

@kunif
Copy link

kunif commented Feb 16, 2022

This is a feature I want even in --auto mode.
It would be nice if the list was displayed before the actual update work was called.

It has less information and less real-time than the progress bar discussed in issue #77, but it seems easy to implement.

And being presented with a list of what will be updated in advance gives you a sense of security that you are aware of the situation in either mode.

If someone responds to this for both modes, would the option name be around --preview?

@jgonggrijp
Copy link
Owner

Implemented by @kunif in #91.

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

No branches or pull requests

3 participants