hyperledger / fabric-x-committer / 23780809492

91%

main: 91%

Build Type

Pull #496

github

Committed by cendhu

Commit Message

restructure CLI command organization

    - Move start commands from cmd/config/cmd.go to cmd/committer/start_cmd.go
    - Consolidate start commands using switch-case pattern, eliminating duplication
    - Create cmd/cliutil package for shared CLI utilities (VersionCmd, SetDefaultFlags)
    - Move test helpers from cmd/config to cmd/cliutil/test_exports.go
    - Remove redundant CLI flags (--endpoint, --metrics-endpoint, --verbose,
      --parallelism, --ledger-path, etc.) that duplicate YAML/env var config
    - Simplify SetDefaultFlags to only register --config flag
    - Remove unused CobraInt/CobraString/CobraBool/CobraDuration flag helpers

    The config package retains config-reading utilities, viper defaults,
    and template definitions. Command wiring is now colocated with the
    binary that uses it.

    Closes hyperledger/fabric-x-committer#495

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #496: restructure CLI command organization and remove redundant CLI flags

Run Details

67 of 83 new or added lines in 3 files covered. (80.72%)

20 existing lines in 5 files now uncovered.

9939 of 10872 relevant lines covered (91.42%)

41885.11 hits per line