Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Replace afp with afpv2 #38

Open
esc opened this issue Dec 29, 2015 · 9 comments
Open

Replace afp with afpv2 #38

esc opened this issue Dec 29, 2015 · 9 comments

Comments

@esc
Copy link
Contributor

esc commented Dec 29, 2015

What are every ones thoughts on this one?

@mriehl
Copy link

mriehl commented Jan 19, 2016

I'd do a 2-step migration:

  • afp uses the new cli experience, but afpv1 kept for backwards compatibility
  • drop afpv1 (at some point)

Step 1 would allow to find out who is affected (eG numerous scripts). If no one complains then step 2 can be done earlier. People whose workflow is disrupted by the change can still keep using afpv1 but will know that they need to adapt their scripts / workflow.

@esc
Copy link
Contributor Author

esc commented Jan 19, 2016

Okay, maybe next tuesday then.

@esc
Copy link
Contributor Author

esc commented Jan 19, 2016

Do you know of any scripts that use afpv1?

@mriehl
Copy link

mriehl commented Jan 19, 2016

We have some, but we can simply change them. I don't have any problems with a hard migration. But who knows what's out there ;)

@esc
Copy link
Contributor Author

esc commented Jan 19, 2016

I'd like to come and see some of those scripts, wasn't expecting anyone to automate on afp-cli.

@schlomo
Copy link
Contributor

schlomo commented Jan 27, 2016

Can we rename the subshell command to just shell? It is shorter and IMHO clear what it will do.

@schlomo
Copy link
Contributor

schlomo commented Jan 27, 2016

Please add an Examples section to the --help command, e.g. eval $(afpv2 export account-name). This will help our users to quickly understand how to use it.

@schlomo
Copy link
Contributor

schlomo commented Jan 27, 2016

Who needs the show subcommand? What use case does it serve? If this is for human consumption then I would suggest a format that is much more friendly on the eyes than KEY=VAL. And you also don't need quoting then :-)

If to keep show then probably JSON and YAML output will be more valuable to enable people to parse it in a machine readable format. jq to the rescue :-)

@schlomo
Copy link
Contributor

schlomo commented Jan 27, 2016

The write subcommand should actually inform about which file was written and maybe warn if the file is world readable. Or offer setting safe perms automatically.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants