glassechidna / stackit

Cross-platform CloudFormation CLI tool for easy synchronous and idempotent stack updates
Apache License 2.0
37 stars 7 forks source link

Better error handling #11

Closed aidansteele closed 6 years ago

aidansteele commented 7 years ago

e.g. https://github.com/glassechidna/stackit/blob/master/stackit/stackit.go#L208

stackExists := err == nil doesn't necessarily mean the stack doesn't exist if there's an error. The error could be lack of permission to describe stacks or something else. Check the exact error message.

And other stuff

aidansteele commented 6 years ago

I'm going to close this on account of that particular code no longer existing, having been replaced by (e.g.):

https://github.com/glassechidna/stackit/blob/8e06a4b4bc6bec07f961183e446313b9e900254e/pkg/stackit/up.go#L63