Code review comment for lp://staging/~ursinha/uci-engine/update-ticket-raises-error

Revision history for this message
Ursula Junque (ursinha) wrote :

> This is a step forward and I'll approve. I would like to make sure we take a
> closer look at the other post and get calls within the cli (and indeed all
> code) and make sure we're not dropping relevant error messages.

All of the posts, gets and patches are covered now, using utils.patch, utils.post and utils.get will always raise_for_status and main cli script will catch and print that. I agree we still need to revisit how we handle exceptions to see if we're not losing important information we could be displaying.

« Back to merge proposal