I'm not opposed @sduc! I'd want to keep it extremely lightweight, keeping the stdlib flag package (such as the simple environment readers that @steviec suggested, with some improvements). There's also a wrinkle with multiple-use switches, in particular the -storage and related functions which have a custom flag accumulator:

Lines 56 to 60 in a192ac3

    
           cliStorage := cli.NewStorage() 
        
           flag.Var(&cliStorage.Storage, "storage", "name of storage backend") 
        
           flag.Var(&cliStorage.DSN, "storage-dsn", "data source name (e.g. connection string or path)") 
        
           flag.Var(&cliStorage.DSN, "dsn", "data source name; deprecated: use -storage-dsn") 
        
           flag.Var(&cliStorage.Options, "storage-options", "storage backend options")

	cliStorage := cli.NewStorage()
	flag.Var(&cliStorage.Storage, "storage", "name of storage backend")
	flag.Var(&cliStorage.DSN, "storage-dsn", "data source name (e.g. connection string or path)")
	flag.Var(&cliStorage.DSN, "dsn", "data source name; deprecated: use -storage-dsn")
	flag.Var(&cliStorage.Options, "storage-options", "storage backend options")

Support environment variable configuration #11

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions