-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shep as a library #258
Labels
Comments
Would also probably make sense to make all calls to shep as a library be quiet by default |
So a few things that I think are still a bit up in the air:
|
As part of this I feel like we should try to remove |
What if the commands accept a `listr` or even more generic `logger`
instance that can be updated as it proceeds? If not provided it can do
nothing.
…On Fri, Jun 16, 2017 at 4:09 PM, Chris Olszewski ***@***.***> wrote:
As part of this I feel like we should try to remove listr calls from the
functions themselves. Big issue with this is shep.deploy as listr is
actually relied on by deploy for control flow
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#258 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAcxaOBIdmg4tRdGMbeDzoldaty2-LYxks5sEoyFgaJpZM4N7OTw>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I think we're pretty close to this being a thing. Just a matter of making sure all the functions return something/can be used without their respective commands being run first.
The text was updated successfully, but these errors were encountered: