-
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
Add plan debugging documentation. #768
Add plan debugging documentation. #768
Conversation
Agree on the move of the "build packages" content. Now we're really telling a story on the two halves of Habitat: The buildtime dev experience and the runtime management/configuration experience. |
@@ -1089,8 +1089,8 @@ attach() { | |||
case "$cmd" in | |||
vars) (set -o posix; set);; | |||
whereami*|\@*) | |||
__attach_whereami "$(echo $cmd \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ha, what was up there?
Yep, this is great! @juliandunn, looks like this needs a rebase then I'll approve |
Fixed a small bug in hab-plan-build where "whereami" doesn't work due to a typo. Moved "build packages" under the create topic; it's really a subset of that. Signed-off-by: Julian C. Dunn <[email protected]>
thanks for the review @fnichol ; |
📌 Commit 720d8b1 has been approved by |
Fixed a small bug in hab-plan-build where "whereami" doesn't work due to a typo. Moved "build packages" under the create topic; it's really a subset of that. Signed-off-by: Julian C. Dunn <[email protected]> Pull request: #768 Approved by: fnichol
☀️ Test successful - travis |
Also: