hyperledger / fabric-x-committer / 23785829163

91%

Build Type

push

github

Committed by web-flow

Commit Message

[cmd] restructure and remove redundant flags (#496)

#### Type of change

- Improvement (improvement to code, performance, etc)
 
#### Description

- 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.

    

#### Related issues

 - resolves hyperledger/fabric-x-committer#495

---------

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

Run Details

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

32 existing lines in 9 files now uncovered.

9915 of 10872 relevant lines covered (91.2%)

42782.86 hits per line