search

f0cal / farm-client

CLI client for FØCAL's cluster of edge devices and AI co-processors
https://f0cal.com
MIT License
1 stars 0 forks source link

Config from env #42

Closed ohinds closed 3 years ago

brianthelion commented 3 years ago

@ohinds What's the UX on this? Want to make sure we're staying consistent.

ohinds commented 3 years ago

@ohinds What's the UX on this? Want to make sure we're staying consistent.

f0cal farm config from-env

brianthelion commented 3 years ago

@ohinds Let's do f0cal farm config set '*' --from-env and I'm good with it.

ohinds commented 3 years ago

@ohinds Let's do f0cal farm config set '*' --from-env and I'm good with it.

I'm fine doing that, but it's certainly not consistent.

brianthelion commented 3 years ago

f0cal farm {class} {verb} {instance} ... is what we've been trying to maintain throughout.

What's your concern?

ohinds commented 3 years ago

f0cal farm {class} {verb} {instance} ... is what we've been trying to maintain throughout.

What's your concern?

Well, that's just not at all consistent with how the config is currently updated. I could see f0cal farm config update --from-env?

Also, the '*' seems unnecessary and error-prone.

brianthelion commented 3 years ago

Yeah, that's legacy. We've been discussing changing it to f0cal farm config update {arg} --value={val}.