Project

General

Profile

Bug #11110

hammer content-view --help having wrong description at remove-version subcommand

Added by Pavel Moravec almost 4 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
trivial
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1242879

Description of problem:
content-view --help prints:

remove-version                Update a content view
update Update a content view

Version-Release number of selected component (if applicable):
rubygem-hammer_cli_katello-0.0.4-14.el6sat.noarch (but seen in upstream as well)

How reproducible:
100%

Steps to Reproduce:
hammer -u admin -p password content-view --help

Actual results:
remove-version has description "Update a content view" / copied from the next subcommand description.

Expected results:
remove-version to have some relevant description.

Additional info:
Upstream PR on the way

Associated revisions

Revision 56058030 (diff)
Added by Pavel Moravec almost 4 years ago

Fixes #11110 - hammer content-view --help wrong remove-content description

remove-content description is wrongly copied from the next sub-command.

The fix just adds proper description to RemoveContentViewVersionCommand to
prevent copying it from the next sub-command.

Signed-off-by: Pavel Moravec <>

Revision 4d05efd8
Added by Eric D Helms almost 4 years ago

Merge pull request #303 from pmoravec/master

Fixes #11110 - hammer content-view --help wrong remove-content descri…

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/303 added
  • Pull request deleted ()

#3 Updated by Eric Helms almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 31
  • Triaged changed from No to Yes

#4 Updated by Eric Helms almost 4 years ago

  • Legacy Backlogs Release (now unused) changed from 31 to 70

#5 Updated by Pavel Moravec almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF