#88 ✓invalid
Simon Rozet

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

    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.

New-ticket 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.

Shared Ticket Bins

People watching this ticket