#1120 Show inheritance flags in list-tag-inheritance output
Merged by tkopecek. Opened by mikem.
mikem/koji show-inheritance-options  into  master

Download 1120.patch

While not often used, tag inheritance can be modified with a few different options (e.g. maxdepth). These options are shown in the taginfo display, but not the list-tag-inheritance display. This patch adds basic indicators to the latter.

Users can be easily confused when these options are in use, so better to make things more visible.

I'm not set on the exact display. I tried a few variations. The 4 character flag display is concise, but doesn't look all that great without alignment. Also, most inheritance lines have no options, so the dots seem repetitive and noisy.

Metadata Update from @mikem:
- Pull-request tagged with: usability

:thumbsup:

Let me show an example so that folks can see that this looks like

Original (without these changes)

$ koji list-tag-inheritance tag_y
tag_y (198)
  └─tag_a (53)

With current changes:

$ koji list-tag-inheritance tag_y
tag_y (198)
  └─tag_a (53)  M...

(indicates the presence of maxdepth option on the inheritance line)

Another example. No inheritance options in the chain.

$ koji list-tag-inheritance tag_a
tag_a (53)
  ├─tag_b (52)  ....
  └─tag_c (193)  ....
     └─tag_d (194)  ....

I'm not loving the way this looks, but I've so far been unable to come up with a better option that is still compact.

should priority be shown?

should priority be shown?

That would make the display even busier, and the ordering already reflects relative priority.

I guess it could be useful, but I don't think it needs to be part of this PR

:thumbsup:

Does this look better?

$ koji list-tag-inheritance tag_a_foo
     tag_a_foo (223)
....  └─tag_a (53)
....     ├─tag_b (52)
....     └─tag_c (193)
....        └─tag_d (194)
$ koji list-tag-inheritance tag_y
     tag_y (198)
M...  └─tag_a (53)

Maybe be, it is better. But both approaches work for me.

1 new commit added

  • move flags to front

Filed #1625 to track this

@julian8628 @breilly

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

I prefer having flags at the start as well.

:thumbsup:

:thumbsup:

Commit 8e548962 fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata