microsoft / bedrock-cli

The CLI for Bedrock
https://microsoft.github.io/bedrock-cli/commands/
MIT License
28 stars 12 forks source link

bedrock `setup` doesn't auto-populate config values from existing config file #52

Open andrebriggs opened 4 years ago

andrebriggs commented 4 years ago

Repro:

  1. Use bedrock init -i to setup an config file at ~/.bedrock/config.yaml

  2. Call bedrock init -i and see that the first prompt already has a value filled in for you

    $ bedrock init -i
    ? Enter organization name
    (myOrg) 
  3. Then call bedrock setup and the first prompt doesn't already have a value filled in for you

    $ bedrock setup
    info:    az cli vesion 2.5.1
    ? Enter organization name
    

I would expect the following

  $ bedrock setup
  info:    az cli vesion 2.5.1
  ? Enter organization name
  (myOrg) 
dennisseah commented 4 years ago

when we design setup command, we hope to provide a way to jumpstart user from ground zero to a functional setup. And the thinking is that there are no spk config file to start with.

andrebriggs commented 4 years ago

@dennisseah makes complete sense. If we are coming from the POV that they will never call bedrock init then we can ignore this