Project

General

Profile

Bug #21371

hammer repository-set enable --help doesn't explain purpose of --new-name

Added by Brad Buckingham almost 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:

Description

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

Description of problem:
calling `hammer repository-set enable --help`, the parameter `--new-name` is listed without explanation.

Version-Release number of selected component (if applicable):
tfm-rubygem-hammer_cli-0.11.0.1-1.el7sat.noarch

How reproducible:
always

Steps to Reproduce:
1. hammer repository-set enable --help

Actual results:
No explanation for --new-name

Expected results:
Short explanation of --new-name

Additional info:
n/a

Associated revisions

Revision 048c526a (diff)
Added by Brad Buckingham almost 4 years ago

fixes #21371 - convert repository-set enable/disable to be Command (#520)

The repository-set enable/disable commands are currently
derived from the UpdateCommand. This is resulting in the
inclusion of a '--new-name' parameter, which is not needed
or used by the commands. As a result, updating them
to be simple Commands so that the parameter is not included.

History

#1 Updated by Brad Buckingham almost 4 years ago

  • Difficulty set to easy
  • Target version set to 225
  • Subject changed from hammer repository-set enable --help doesn't explain purpose of --new-name to hammer repository-set enable --help doesn't explain purpose of --new-name

#2 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/520 added

#3 Updated by Brad Buckingham almost 4 years ago

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

#4 Updated by Eric Helms almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 250

#5 Updated by The Foreman Bot almost 4 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/531 added

#6 Updated by Andrew Kofink almost 4 years ago

  • Pull request deleted (https://github.com/Katello/hammer-cli-katello/pull/531)

#7 Updated by The Foreman Bot almost 4 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/531 added

Also available in: Atom PDF