Feature #12999

Rename add-override-value to add-matcher under sc-param.

Added by Jitendra Yejare over 2 years ago. Updated 5 months ago.

Status:Closed
Priority:Normal
Assigned To:Rahul Bajaj
Category:Other commands
Target version:Foreman - Team Anurag Iteration 20
Difficulty:easy Bugzilla link:1295780
Found in release: Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/327
Story points-
Velocity based estimate-
Release hammer-cli-foreman-0.12.0Release relationshipAuto

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1295780
Description of problem:
Rename the sub-command add-override-value to add-matcher under sc-param.

add-override-value tends to set the default value by overriding the parameter which was not overrided earlier. And as per discussion with Dev and behaviour of this sub-command its better to rename it to add-matcher.

Version-Release number of selected component (if applicable):
foreman-1.11.0-0.develop.201512111432git98f6ca5.el7.noarch

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:

Expected results:
Rename add-override-value to add-matcher under sc-param.

Additional info:

Associated revisions

Revision 2f40c76e
Added by Rahul Bajaj 5 months ago

Fixes #12999 - Rename add-override-value to add-matcher

History

#1 Updated by Dominic Cleal over 2 years ago

  • Project changed from Foreman to Hammer CLI
  • Category set to Foreman commands (obsolete)

#2 Updated by Tomáš Strachota over 2 years ago

Please note that we need to keep original 'add-override-value' command in place for backwards compatibility. It can probably be hidden alias to the new command with a deprecation warning.

#3 Updated by Tomáš Strachota 10 months ago

  • Category changed from Foreman commands (obsolete) to Other commands
  • Difficulty set to easy

#4 Updated by Rahul Bajaj 10 months ago

  • Assigned To set to Rahul Bajaj

#5 Updated by Rahul Bajaj 9 months ago

  • Target version set to Team Anurag Iteration 19

#6 Updated by The Foreman Bot 9 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/327 added

#7 Updated by Kavita Gaikwad 9 months ago

  • Target version changed from Team Anurag Iteration 19 to Team Anurag Iteration 20

#8 Updated by Rahul Bajaj 5 months ago

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

#9 Updated by Martin Bacovsky 5 months ago

  • Release set to hammer-cli-foreman-0.12.0

Also available in: Atom PDF