-
Notifications
You must be signed in to change notification settings - Fork 315
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
Adds cli output to Try Habitat feature #483
Conversation
Signed-off-by: Ryan Keairns <[email protected]>
Signed-off-by: Ryan Keairns <[email protected]>
By analyzing the blame information on this pull request, we identified @cwebberOps to be a potential reviewer |
@smith @juliandunn this is the PR for Try Hab content. Let me know what you think, thanks! |
@magwalk you may have thoughts... can you please take a look at the styling. |
@ryankeairns you decided not to go with Otherwise, 👍 and if @magwalk has no comments by tomorrow I'll r+ this |
@juliandunn I started with The other reason is that I was getting permission errors with |
@ryankeairns nah let's just go with this for now to get the "Try Habitat" experience out there. I can help you debug |
📌 Commit 5d1503c has been approved by |
Signed-off-by: Ryan Keairns <[email protected]> Pull request: #483 Approved by: juliandunn
Signed-off-by: Ryan Keairns <[email protected]> Pull request: #483 Approved by: juliandunn
☀️ Test successful - travis |
Signed-off-by: Ryan Keairns <[email protected]> Pull request: #483 Approved by: juliandunn
Signed-off-by: Ryan Keairns <[email protected]> Pull request: #483 Approved by: juliandunn
With things taking shape, I was able to run all of the commands (except -strategy and -bind) and get the actual output into the steps. This commit also adds some tab-style buttons to switch between terminal windows on a couple of steps and add a back link to view previous step.