Make sure use of `ensure` doesn't hide errors
Reported by Simon Rozet | February 14th, 2009 @ 01:28 AM | in 0.1.9
20:20 sr if an exception is raised, it'll go straight to the ensure block...without setting @build 20:20 foca oh 20:21 sr so, let's say something fail in SCM and kaboum! 20:21 foca ok, then: a) let's change /dev/null for /tmp/integrity.log or something 20:21 sr that's what I meant by "it hides errors" 20:21 sr a) done 20:21 foca b) put a `raise` at the end of the ensure block :) 20:45 sr huum, foca_ I think we want to use a transaction and rollback in an exception is raised 20:45 sr or something liek that 20:47 sr foca_, http://gist.github.com/64063 20:47 sr that's what happen if I remove the ensure 20:49 sr sooo, I am not doing anything wrt this for now
Comments and changes to this ticket
-
Simon Rozet February 16th, 2009 @ 05:18 AM
- State changed from “new” to “invalid”
- Assigned user changed from “ronin-278 (at lighthouseapp)” to “Simon Rozet”
This actually is a non-issue :)
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Automated continuous integration server that doesn't suck.
Simon Rozet