search

iceberg-project / ICEBERG-middleware

This repo contains the ICEBERG middleware as it is agreed by the members of the ICEBERG project.
MIT License
2 stars 1 forks source link

Simplify argument parsing in ICEBERG command #8

Closed iparask closed 5 years ago

iparask commented 5 years ago

Instead of having a complicated arg parsing mechanism, fully use arg parse capabilities for that.

Proposal 1: Reduce all arguments to a single level, regardless the use case. The command will be very close to what it is right now, but all arguments could be set. In the end based on the use case some may be use some not.

Proposal 2: Use argparse commands to specify the use case and as a result the rest of the parameters. it will look like:

iceberg usecase [<args>]

where use case will be one of the supported usecases.

What do you think is preferable?

andre-merzky commented 5 years ago

Note that argparse supports subcommands

iparask commented 5 years ago

Arg parse commands it is then! Thank you @mturilli, @bspitzbart, @AymenFJA