Sprint 4 #7
Sprint 4 #7
Conversation
✅ Deploy Preview for legendary-cat-a62c0e ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Как я понял из обсуждения в пачке с наставником, тест на pre-commit не проходит из-за бага. Для него всё установлено и работает. Если уже дело не в баге, то буду исправлять. |
There was a problem hiding this comment.
Поздравляю! Ваша работа принята.
Вы отлично потрудились.
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что в
pre-commitзапускаются и тесты, и линтеры - Все тесты (24 шт) проходят успешно
npm auditпоказывает ноль уязвимостей- Проект отлично выглядит и функционирует
Можно лучше
- Попробуйте максимально избавиться от типов
anyв коде. Хотя бы часть из них можно заменить наunknown, если совсем не получается. Чаще всего помогает использование дженериков. Подробнее тут https://habr.com/ru/companies/tbank/articles/588655/ - Для линтинга на прекоммите можно воспользоваться библиотекой lint-staged . Она позволяет осуществлять линтинг только тех файлов, которые изменились
| "@types/mocha": "^10.0.7", | ||
| "@types/node": "^22.5.0", |
There was a problem hiding this comment.
В devDependencies указывается то, что необходимо для сборки и разработки приложения, а в dependencies нужно указывать то, что необходимо непосредственно для работы приложения (express, uuid). Все линтеры, тесты, типы и тд нужно только в devDependencies указывать, чтобы зря не раздувать кол-во пакетов в собранном для деплоя приложении.
| "dev": "vite --port 3000", | ||
| "build": "vite build", | ||
| "preview": "vite preview", | ||
| "start": "vite build && vite preview --port 3000", | ||
| "eslint": "npx eslint" | ||
| "eslint": "npx eslint", |
There was a problem hiding this comment.
Можно лучше
Для линтинга на прекоммите можно воспользоваться библиотекой lint-staged . Она позволяет осуществлять линтинг только тех файлов, которые изменились
No description provided.