Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

Simplify CLI usage by deprecating readvars #133

Closed
azlam-abdulsalam opened this issue Aug 25, 2020 · 2 comments · Fixed by #136
Closed

Simplify CLI usage by deprecating readvars #133

azlam-abdulsalam opened this issue Aug 25, 2020 · 2 comments · Fixed by #136
Assignees
Labels
enhancement enhancement to existing feature
Milestone

Comments

@azlam-abdulsalam
Copy link
Contributor

sfpowerscripts cli supports output variables, which are written to the environment of the host. However to consume this, a source readvars has to be issued before any other command. This makes the scripts verbose and defeat the purpose.

sfpowerscripts cli should store this output variable in its own environment config, and all tasks when provided with a non dereferenced variable, should check the store and figure the value. It should also print the resolved variables

@aly76
Copy link
Contributor

aly76 commented Aug 29, 2020

I propose that we keep the readvars script for now. It may still be useful for power users, who want access to output variables at the terminal layer.

@azlam-abdulsalam
Copy link
Contributor Author

Sure we can keep it, but lets remove it from our main readme and add us a subsection

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement enhancement to existing feature
Projects
None yet
2 participants