cli icon indicating copy to clipboard operation
cli copied to clipboard

altsrc requires flag names

Open eyudkin opened this issue 5 years ago • 4 comments

What problem does this solve?

For now altsrc requires to name my flags using dots to find them into multi-level config files. I.e. to find a flag "level" in the following config:

{
  "infrastructure": {
    "logger": {
      "level": 10
    }
  }
}

..i need to name my flag "infrastructure.logger.level" without exceptions. But what if I don't need it? For my cli interface I want its name to be "--log-level", because I don't need any nested things there. So, I want to be able name flags as I want and configure paths as I want too, separately.

Solution description

Something like:

altsrc.NewIntFlag(&cli.IntFlag{
	Name: "log-level",
	Usage: "Logger level (0 - debug, 1 - info, ...)",
	Value: 1,
}, "my.path.to.flag.in.config"), // added customizable parameter here

Describe alternatives you've considered

Using aliases is not an option there because this ugly long "infrastructure.logger.level" name will be displayed at help screen anyway.

eyudkin avatar Jan 23 '20 14:01 eyudkin

Heya! We are planning on completely rewriting altsrc for v3, so its unlikely that a maintainer will get to this issue! I'm marking it as help wanted for any other contributor that wants to help, though 🙏

coilysiren avatar Mar 30 '20 23:03 coilysiren

Ah, I just re-read the issue. It looks like this is a feature request? Since altsrc is about to be rewritten, we aren't currently accepting feature requests for it.

coilysiren avatar Mar 30 '20 23:03 coilysiren

This issue or PR has been automatically marked as stale because it has not had recent activity. Please add a comment bumping this if you're still interested in it's resolution! Thanks for your help, please let us know if you need anything else.

stale[bot] avatar Jun 28 '20 23:06 stale[bot]

Closing this as it has become stale.

stale[bot] avatar Jul 29 '20 00:07 stale[bot]