Change config semantics to args and full key path

[feedmeapples]

What was changed

changes config feature semantics from flags to args

tctl config set --namespace accounting

to

tctl config set env.local.namespace accounting (or tctl config set namespace accounting for short in a new PR)

Why?

• moving towards allowing to configure the default value for any flag and not just the ones whitelisted by tctl config set command
• consistency with other tools such as git and kubectl

Checklist

1. Closes

2. How was this tested:

3. Any docs updates needed?

[bergundy]

When is it valid to omit the value?

[feedmeapples]

was thinking of booleans such as tls-disable-host-verification

[feedmeapples]

so tctl config set tls-disable-host-verification would mean it being set as true. That's similar to when it is passed as a flag

[bergundy]

Hmm.. how do we deal with booleans and numbers in general?

[feedmeapples]

made them to work with the standard urfave/cli syntax #257

[bergundy]

For someone like me who's not familiar with the codebase a PR description would go a long way.
Also it doesn't make sense to me not to have any test coverage for this.

[feedmeapples]

For someone like me who's not familiar with the codebase a PR description would go a long way. Also it doesn't make sense to me not to have any test coverage for this.

right, missed adding description after removing the draft tag