When I see code like this I will assume that it's going to check every condition:

        if !complete {
            // do something
        }
        if stat, ok := r.Object.(*api.Status); ok && stat.Code != 0 {
            // do something
        }
        if r.Created {
            // do something
        }

The recommended way means that I can't trust my assumption and that I'll have to read the entire function and examine the `//do something`s instead