Refactor #19574

Option validator should use option sources

Added by Tomáš Strachota about 1 year ago. Updated 3 months ago.

Status:Ready For Testing
Priority:Normal
Assigned To:Tomáš Strachota
Category:Hammer core
Target version:Foreman - Team Marek Iteration 25
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/hammer-cli/pull/242
Story points-
Velocity based estimate-

Description

In https://github.com/theforeman/hammer-cli/commit/5bf50733c576963a316b6978954a1b775b810a1c Hammer introduced option sources that enable reading option values from additional providers. Default options have been turned into an option source too. Validator currently reads defaults directly. It should use option sources instead, otherwise it will prevent a command to start when required options come from some additional option source.

History

#1 Updated by The Foreman Bot about 1 year ago

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

#2 Updated by Marek Hulán 11 months ago

  • Target version changed from Team Marek Iteration 15 to Team Marek Iteration 16

#3 Updated by Marek Hulán 11 months ago

  • Target version changed from Team Marek Iteration 16 to Team Marek Iteration 17

#4 Updated by Marek Hulán 10 months ago

  • Target version changed from Team Marek Iteration 17 to Team Marek Iteration 18

#5 Updated by Marek Hulán 10 months ago

  • Target version changed from Team Marek Iteration 18 to Team Marek Iteration 19

#6 Updated by Marek Hulán 9 months ago

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

#7 Updated by Marek Hulán 8 months ago

  • Target version changed from Team Marek Iteration 20 to Foreman - Team Marek Iteration 21

#8 Updated by Marek Hulán 7 months ago

  • Target version changed from Foreman - Team Marek Iteration 21 to Foreman - Team Marek Iteration 22

#9 Updated by Marek Hulán 7 months ago

  • Target version changed from Foreman - Team Marek Iteration 22 to Foreman - Team Marek Iteration 23

#10 Updated by Marek Hulán 6 months ago

  • Target version changed from Foreman - Team Marek Iteration 23 to Foreman - Team Marek Iteration 24

#11 Updated by Marek Hulán 5 months ago

  • Target version changed from Foreman - Team Marek Iteration 24 to Team Marek Iteration 25

Also available in: Atom PDF