canonical / test_observer

Flutter based dashboard for visualising SRU regression test results
2 stars 3 forks source link

Only consider latest builds when checking if signoff is allowed #130

Closed omar-selo closed 7 months ago

omar-selo commented 7 months ago

Looks like the previous PR to fix the issue of "when checking if an artefact status change is allowed or not we shouldn't take into consideration old hidden builds." doesn't work. Main reason is that turns out itertools.groupby requires input to be sorted. This PR should fix the issue, and I've consolidated this logic in one place.