Skip to content

Feat/icons generation reproducibility check#716

Open
comfrt1k wants to merge 4 commits into
feat/upgrade-depsfrom
feat/icons-generation-reproducibility-check
Open

Feat/icons generation reproducibility check#716
comfrt1k wants to merge 4 commits into
feat/upgrade-depsfrom
feat/icons-generation-reproducibility-check

Conversation

@comfrt1k

Copy link
Copy Markdown
Contributor

Таска

Как проверять

  1. Контекст: локальная проверка воспроизводимости генерации @atls-ui-admin/icons
    Действие: выполнить yarn workspace @atls-ui-admin/icons check-generated
    Ожидаемый результат: команда завершается успешно, выводит Generated icons are up to date, рабочее дерево после запуска остаётся чистым

  2. Контекст: негативная проверка 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

  3. Контекст: сборка затронутых пакетов
    Действие: выполнить yarn workspace @atls-ui-admin/icons build и yarn workspace @atls-ui-generators/icons build
    Ожидаемый результат: обе сборки завершаются успешно

Пруфы

Details image image image

@TorinAsakura TorinAsakura left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

кажется слой ui-admin/icons/scripts/ давно уже перерос в lib как минимум, нужна реорганизация

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants