On Wed, Jul 17, 2019 at 01:53:52PM +0200, Andrea Bolognani wrote: > Since the user is explicity asking for information to be > displayed, we should present it to them whether or not quiet > mode is enabled. The reason why I honoured the "quiet" flag in this command was because it serves as a "does the repo exist" check, and when used for that purpose avoids need to redirect stdout to /dev/null. > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx> > --- > guests/quayadmin | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/guests/quayadmin b/guests/quayadmin > index 5dc5eff..980fc1b 100755 > --- a/guests/quayadmin > +++ b/guests/quayadmin > @@ -84,8 +84,8 @@ def run_show_repo(args): > return 1 > > info = res.json() > - if not args.quiet: > - print("{}/{}: {}".format(args.namespace, args.repo, info["description"])) > + > + print("{}/{}: {}".format(args.namespace, args.repo, info["description"])) Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list