When updating the branch status, update the appropriate commit instead of always updating the head

[kylegibson]

Currently whenever the branch status is updated to indicate whether tests are passing or failing, we always mark the head commit. This is bad, because it can result in code being merged in and deployed that hasn't been tested.

When hubot receives instructions to run the tests, save the commit. As test results are received, retrieve the saved commit and update it.

Also:

• Don't mark the build as pending until the ticket job actually starts.
• Update the documentation to explain some of the assumptions, and add some more sane defaults
• Dynamically grab the number of downstream jobs so that we don't have to update an environment variable
• As soon as one of the downstream jobs fails, mark the branch as failing. No need to wait for all of the rest.

[winhamwr]

Updated the spec to cover the rest of the stuff I did.

[kylegibson]

We should consider making a flow diagram to assist in grokking how this works.

[winhamwr]

Any other review stuff I can fix in the master branch. We were already using this live, anyway :)

Also, I can't read the trello card and thought you had already moved this before I clicked merge. derp

[kylegibson]

Nice