mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
ci(process): ask the reviewer about the wall key invariant
Issue: #259 User-Visible: no
This commit is contained in:
@@ -455,10 +455,13 @@ jobs:
|
||||
инварианты модели (#254): `npm test` уже гоняет их на всех моделях
|
||||
проекта, а на конкретной конфигурации они проверяются командой
|
||||
`npm run invariants -- --config <экспорт или ответ config/get>`.
|
||||
Два вопроса, на которые они отвечают, и оба уже стоили продукту
|
||||
дефектов: не исчезла ли запись толщины (#253) и разрешима ли каждая
|
||||
ссылка (#244, #252). Если задача меняет геометрию, а инварианты в
|
||||
отчёте не названы — это непрогнанный гейт, а не мелочь.
|
||||
Три вопроса, на которые они отвечают, и все три уже стоили
|
||||
продукту дефектов: не исчезла ли запись толщины (#253), разрешима ли
|
||||
каждая ссылка (#244, #252) и равен ли ключ записи толщины ключу
|
||||
решёточного ребра (#258, #259). Последний сравнивает строки без
|
||||
допусков: сдвиг ключа на один шаг решётки равен допуску первых двух,
|
||||
поэтому они на нём промахиваются. Если задача меняет геометрию, а
|
||||
инварианты в отчёте не названы — это непрогнанный гейт, а не мелочь.
|
||||
|
||||
По необходимости, и «необходимость» определяется diff'ом и AC:
|
||||
- браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс те,
|
||||
|
||||
Reference in New Issue
Block a user