mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
ci: name check-docs in the review gate set
Issue: #237 User-Visible: no
This commit is contained in:
@@ -353,12 +353,20 @@ jobs:
|
|||||||
Всегда, они дешёвые, и в повторном раунде тоже: код изменился,
|
Всегда, они дешёвые, и в повторном раунде тоже: код изменился,
|
||||||
а стоят они минуты:
|
а стоят они минуты:
|
||||||
`npx tsc --noEmit`, `npm test`, `npm run build` со сверкой трёх
|
`npx tsc --noEmit`, `npm test`, `npm run build` со сверкой трёх
|
||||||
копий бандла.
|
копий бандла. Плюс `node scripts/check-docs.mjs`, если diff трогает
|
||||||
|
`src/**`: отпечаток скриншотов документации считается по всему
|
||||||
|
`src/**`, поэтому любая правка фронтенда делает его устаревшим —
|
||||||
|
выбирать тут нечего. Пропуск этого шага в #230 и #234 оставил `dev`
|
||||||
|
с красным job `docs` до следующей задачи (#237).
|
||||||
|
|
||||||
По необходимости, и «необходимость» определяется diff'ом и AC:
|
По необходимости, и «необходимость» определяется diff'ом и AC:
|
||||||
- браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс
|
- браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс
|
||||||
относящиеся к тронутым поверхностям. Их 127; прогон всех уместен
|
относящиеся к тронутым поверхностям. Их 163; прогон всех уместен
|
||||||
только когда задача действительно задевает всё;
|
только когда задача действительно задевает всё. Выбирать по теме
|
||||||
|
недостаточно: регресс #234 поймал `smoke_wall_junctions`, который
|
||||||
|
по названию про стыки стен, а не про толщину отрезка. Надёжнее
|
||||||
|
grep — искать в `demo/smoke_*.mjs` имена изменённых функций и
|
||||||
|
полей, и прогонять найденное;
|
||||||
- `npm run golden:verify` — если diff может изменить видимый
|
- `npm run golden:verify` — если diff может изменить видимый
|
||||||
результат: рендер, геометрия, стили, слои;
|
результат: рендер, геометрия, стили, слои;
|
||||||
- `python -m pytest tests_backend -q` — если тронут
|
- `python -m pytest tests_backend -q` — если тронут
|
||||||
|
|||||||
Reference in New Issue
Block a user