-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix shrinkwrap and clean up docs #62
Conversation
Since a bunch of the commands are now returning Promises, make the outer wrapper Promise-aware so that we can set appropriate exit codes when we fail.
can you add docs for slt-release, too, please |
This PR doesn't affect |
slt-release docs are not current - they don't mention the new behaviour you added to merge back to master without merging shrinkwrap
|
This was added in d1ffb20 but without adequate documentation around the non-obvious behaviour.
@slnode test please |
Addresses a couple bugs reported as comments on #56.