Stop showing completed try builds

[Mark-Simulacrum]

The UI for rust-lang/rust frequently fills up with numerous "success" at the top of the page.

There's sort of two bugs here. First, we don't currently keep whether a build is try or r+ in the DB, AFAICT, which means that these aren't displayed as success (try) after bors reboots.

The easy fix here is to just never show successful builds -- I'm not sure that they are important enough to be so prominent in the UI, since there's nothing that can really be done with them.

[kennytm]

Better just move them below the "approved" PRs instead of hiding.

[Mark-Simulacrum]

What purpose is there to showing completed builds? If there's not a concrete reason I disagree that we should move them rather than hiding them.

[kennytm]

The same reason we show PRs which have no status.

[Mark-Simulacrum]

Ah -- I guess I should clarify. I meant that the completed PR would have no additional label (i.e., be exactly the same as any not-yet r+-ed PR, both in terms of order and place in the list).

[bryanburgers]

Along the same lines, I wonder if stuffing failures way at the bottom is also counter-productive. If anything, those are PRs that are getting more attention, so they should be nearer the top than PRs that haven’t been reviewed at all.