Feat/icons generation reproducibility check#716
Conversation
TorinAsakura
left a comment
TorinAsakura
left a comment
There was a problem hiding this comment.
кажется слой ui-admin/icons/scripts/ давно уже перерос в lib как минимум, нужна реорганизация
|
TorinAsakura
left a comment
TorinAsakura
left a comment
There was a problem hiding this comment.
Архитектурный контур не проходит: проверочный сценарий и роли файлов разложены не по владельцу ответственности
There was a problem hiding this comment.
у тебя выше уже есть file, кажется что outdated это просто состояние того же файла, нейминг снова занимается ересью, как и содержимое
There was a problem hiding this comment.
констрейнты как раз таки задают вопрос is-**, то есть в данном случае is-directory-required ну и далее также
| const isMissingError = (error: unknown): boolean => | ||
| typeof error === 'object' && | ||
| error !== null && | ||
| 'code' in error && | ||
| (error as { code?: unknown }).code === 'ENOENT' |
There was a problem hiding this comment.
- крайне странная и запутанная конструкция, выглядит так, что проверка на ошибку проверяет проверку на ошибку…
- если это is то выглядит как констрейнт, хотя, опять же, по коду я вообще не понимаю что это и для чего
- если же это еррор (в чём я чот сомневаюсь) то как будто бы ему тут не место
There was a problem hiding this comment.
теперь чек-дженерейтед переехал сюда?) И почему это "скрипт"?)
2 participants
Таска
Как проверять
Контекст: локальная проверка воспроизводимости генерации
@atls-ui-admin/iconsДействие: выполнить
yarn workspace @atls-ui-admin/icons check-generatedОжидаемый результат: команда завершается успешно, выводит
Generated icons are up to date, рабочее дерево после запуска остаётся чистымКонтекст: негативная проверка drift сгенерированных icon-компонентов
Действие: временно изменить любой файл в
ui-admin/icons/src/icons, например заменить одинcurrentColorвbag.icon.tsx, затем выполнитьyarn workspace @atls-ui-admin/icons check-generatedОжидаемый результат: команда падает с ошибкой
Generated files are outdatedи указывает изменённый файл какcontent differsКонтекст: сборка затронутых пакетов
Действие: выполнить
yarn workspace @atls-ui-admin/icons buildиyarn workspace @atls-ui-generators/icons buildОжидаемый результат: обе сборки завершаются успешно
Пруфы
Details