cmd: stacker with no args shouldn't crash #268
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We were getting something like:
panic: runtime error: slice bounds out of range [:2] with capacity 1
goroutine 1 [running]:
main.main.func3(0xc0005ca000)
/stacker-tree/cmd/main.go:289 +0xb55
github.com/urfave/cli.(*App).Run(0xc0005b4000, {0xc000110200, 0x1, 0x1})
/stacker-tree/.build/gopath/pkg/mod/github.com/urfave/cli@v1.22.5/app.go:264 +0x56d
main.main()
/stacker-tree/cmd/main.go:297 +0x120b
derp :)
Signed-off-by: Tycho Andersen tycho@tycho.pizza