-
-
Notifications
You must be signed in to change notification settings - Fork 269
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
status: add icons to indicate package updatability #2906
status: add icons to indicate package updatability #2906
Conversation
486551d
to
7350b8c
Compare
I think this is an appropriate level of noise to make. Originally I thought of mimicking the icon for the "not downloaded warning" but maybe having it close to the versions, as done here, makes more sense. I wonder if having a short hint towards |
I think this would be a good addition! |
Added!
|
I was thinking we would have a hint for either symbol. |
Ok. How about this
|
I like it! |
Missing a colon after |
Info is colored green, like printpkgstyle |
29b9a33
to
9c7b37e
Compare
Is the last commit one reformat too far? It aligns the info past the version
|
774d091
to
e440b8e
Compare
9f53af5
to
dd79a76
Compare
I'm happy with this now. I think the new formatting of the |
Can't say I am an immediate fan of the aligning but it might be one of those things that take get used to. When I thought about a feature like this, I was thinking of putting the icon in the left gutter (which we already use to show the icon when a package is not downloaded). |
yeah, that's definitely viable. Without the tips I thought it needed to be beside the version number to make it clearer, but with the tips I think the gutter is a good place for it, like the not downloaded icon & tip. I'll undo the reformatting |
Perhaps similarly to #2866 in the diff mode when there are no changes we could also list packages that are held back?
currently you'd just get
|
Seems reasonable to me. |
Although, would that make the view for |
Perhaps just
|
Nice! |
Presumably it's possible to get a situation where, after you do E.g. if there are two optima, and the resolver has to pick one. In that case, there would be "some packages that have new versions available", except that they aren't being held back, right? Presumably we should still print some kind of message in that case? |
Maybe: "some packages have new versions available, but it is not possible to update to those new versions without downgrading some other packages". |
I think that's rare and |
Added to
|
acddebe
to
e697c9e
Compare
looks a bit misaligned. Should the |
Maybe the I'll look into that alignment bug |
Actually both upgradable and heldback lines in the But, I think the right answer might emerge after using it? We could go with yellow and see? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's try with this iteration for now. Always possible to tweak based on feedback. Great job!
Thank you Ian! |
Adds two icons to the status report:
⌃
for packages where a newer version can be installed⌅
for packages where a newer version exists, but cannot be installedAlso adds padding to align the version numbers to make things a bit tidier?If the user wants more information they can use
pkg> status --outdated
I don't see a significant increase in execution time
master:
0.004629 seconds (8.33 k allocations: 722.516 KiB)
this PR:
0.006784 seconds (8.40 k allocations: 771.539 KiB)
Related issues
#1655
#2682
Todo
Edit: