Wrap strconv.ParseInt parse errors to provide easier debugging.

josmo / drone-ecs

Drone plugin for triggering Amazon EC2 Container Service (ECS) deployments

Apache License 2.0

30 stars 41 forks source link

Wrap strconv.ParseInt parse errors to provide easier debugging. #31

Closed jrasell closed 5 years ago

jrasell commented 5 years ago

When a failure occurs performing strconv.ParseInt, the error message is printed and propagated without any manipulation. This means Drone will display the error message: strconv.ParseInt: parsing "": invalid syntax which is not very helpful.

This change wraps the parse error with additional information regarding the param which failed so operators can debug easier.

josmo commented 5 years ago

Thanks @jrasell