integrity/integrity — pull request #140 by richmeyers (2011-10-20 04:31:00+00:00)
Title: When return value of save/update is not checked, raise exceptions on failure

Related to #78. Integrity performs save/update calls in several places without checking their return values. Notably this happens when gathering a commit's metadata during build. If a commit message is too long, save fails silently and integrity fails later when it tries to update build status, due to the build's child (commit) being dirty.

The change here causes unchecked save/update calls to raise exceptions on failure, such that the problems are reported against the code actually responsible for them.

Proper handling of long commit messages is in #130.