mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
Правка #388 уронила dev, и виновата подмена предиката. Я взял доказательством `conclusion=success`, то есть оправдательный вердикт. Гейтам диапазона нужен другой факт: судили ли этот коммит вообще. Упавший прогон коммит судил — вердикт вынесен, автор его видел; переоткрывать такой коммит диапазоном не надо. Не судил только отменённый. Цена ошибки была наглядной. Backend на dev красный несколько дней по своей причине (test_ha_import_export), успешных прогонов нет вовсе, поэтому база уезжала на десятки коммитов назад. На83d646c— docs-коммите — гейт «новый код не добавляет any» предъявил 5 чужих находок изe4e1e370и8d431d6dи уронил frontend. Ровно тот сценарий обвинения невиновного, ради которого делался #386, только устроенный мной. Теперь `judgedShas` считает судимыми завершённые прогоны с любым вердиктом, кроме cancelled, а `greenShas` остаётся для классификации (#387): там вопрос другой — доказано ли, что тяжёлые гейты на этом дереве ПРОШЛИ. Два вопроса, два предиката, и путать их дорого. Запрос к API стал `status=completed` — надмножество, нужный предикат применяет скрипт. Issue: #388 User-Visible: no