Project

General

Profile

Actions

Refactor #19574

closed

Option validator should use option sources

Added by Tomáš Strachota almost 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Hammer core
Target version:
-
Difficulty:
Triaged:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

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.

Actions #1

Updated by The Foreman Bot almost 7 years ago

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

Updated by Marek Hulán almost 7 years ago

  • Target version changed from 1.13.4 to 1.14.0
Actions #3

Updated by Marek Hulán almost 7 years ago

  • Target version changed from 1.14.0 to 1.14.3
Actions #4

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.14.3 to 1.17.0-RC2
Actions #5

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.17.0-RC2 to 1.18.0-RC2
Actions #6

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.18.0-RC2 to 214
Actions #7

Updated by Marek Hulán over 6 years ago

  • Target version changed from 214 to 1.16.0-RC2
Actions #8

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.16.0-RC2 to 1.16.0-RC1
Actions #9

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.16.0-RC1 to 1.16.2
Actions #10

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.16.2 to 1.16.1
Actions #11

Updated by Marek Hulán over 6 years ago

  • Target version changed from 1.16.1 to 238
Actions #12

Updated by Anonymous over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF