Closed azinit closed 2 years ago
Добавить базовые тесты на "запускаемость" cli
Хочу баш скрипт говорили они =))
Добавить базовые тесты на "запускаемость" cli
Хочу баш скрипт говорили они =))
Ну по сути это просто cd packages/cli && npm run build && node ./dist/cli.js
добавить в пайплайн)
И если процесс не упадет - то считай все ОК
(либо же если сможешь чтобы на уровне билда с роллапом ругалось сразу - то тоже приемлемо - главное "глупые" ошибки отлавливать сразу перед публикацией)
Problem
Протестил cli перед публикацией. Есть критичные недоработки, что не проверили при влитии)
[ ] Добавить базовые тесты на "запускаемость" cli Не нужно упарываться в тесткейсы, достаточно просто проверить что
npm build + npm start
запускается и не падает для cli Т.к. во время отладки при сборке даже ничего не упало, хотя код в мастере по факту не запускался 🤷♂️ (пришлось фиксить налету)[ ] Добавить визуальную итеративность установке В первом прототипе cli каждая зависимость устанавливалась отдельно, что было визуально приятно смотреть - "Минимум лишней инфы. И актуальное обновление статусов" Почему-то после рефакторинга теперь все это устанавливается в одну строчку - и для плагинов и для TS Нужно вернуть "как было"
[ ] Ложно позитивная установка TS зависимостей Проект [eslint-config] явно детектится как не TS-овый (см. консоль на скрине) Но при этом хоть и в
ui
передается false-значение, установка TS зависимостей все равно происходит