Code review comment for lp:~adeuring/launchpad/bug-494075

Revision history for this message
Brad Crittenden (bac) wrote :

Hi Abel,

Thanks for this fix. Anything to make chasing down OOPs easier is great.

In your actual diff at http://paste.ubuntu.com/337502/ you need to fix the typos in the docstring:

""A variant o TimeoutError hat reports the original PostgreSQL error."""

Looks like you were thinking faster than your fingers would go.

Other than that it's fine.

review: Approve (code)

« Back to merge proposal