PR status only take the first check into account

[bmispelon]

I was confused today because my PR was marked as "build: success" on Trac, but it had fail on github.

Investigating the issue, I discovered that the javascript that figures out the status of a PR only takes the first check into account:

code.djangoproject.com/trac-env/htdocs/tickethacks.js

Line 124 in a8774ab

build_state = data[0].state;

Considering we have around 50 checks, I don't think this logic is accurate.

I would suggest that the status should be "building" if any individual check has status=building, "error" if at least one check has status=error, and "success" only if all checks have status=success (not sure if there are other statues to take into account)

[dev-akshaygupta]

Hey @bmispelon,
I'm eagerly looking for contributing to the Open Source Projects. This will be my first.
I would like to contribute to this one.

[dev-akshaygupta]

@bmispelon
Any feedback on the PR is most welcome, thanks!