Sprint 4 #68
Sprint 4 #68
Conversation
There was a problem hiding this comment.
Поздравляю! Ваша работа принята.
Вы отлично потрудились.
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что в
pre-commitзапускаются и тесты, и линтеры - Все тесты (15 шт) проходят успешно
npm auditпоказывает ноль уязвимостей- Проект отлично выглядит и функционирует
Удачного дальнейшего обучения.
Можно лучше
- Если типизация повторяется в каждом методе (функции), то нужно типизировать сам метод (функцию), а не дублировать типизацию аргументов. Код станет чище
- Для линтинга на прекоммите можно воспользоваться библиотекой lint-staged . Она позволяет осуществлять линтинг только тех файлов, которые изменились
- И теперь нужно
forkубрать, склонировать себе репозиторий, чтобы он стали именно Ваш.
| "lint:css": "stylelint \"**/*.css\"", | ||
| "test": "mocha", | ||
| "prepare": "husky install", | ||
| "pre-commit": "npm run lint:ts && npm run lint:css && npm run test" |
There was a problem hiding this comment.
- Отлично, что в
pre-commitзапускаются и тесты, и линтеры
| "pre-commit": "npm run lint:ts && npm run lint:css && npm run test" | ||
| }, | ||
| "dependencies": { | ||
| "@types/express": "^4.17.21", |
There was a problem hiding this comment.
"@types/express" нужно перенести в devDependencies
В devDependencies указывается то, что необходимо для сборки и разработки приложения, а в dependencies нужно указывать то, что необходимо непосредственно для работы приложения (express, uuid). Все линтеры, тесты, типы и тд нужно только в devDependencies указывать, чтобы зря не раздувать кол-во пакетов в собранном для деплоя приложении.
| "lint:ts": "eslint . --ext .ts,.js", | ||
| "lint:css": "stylelint \"**/*.css\"", |
There was a problem hiding this comment.
Можно лучше
Для линтинга на прекоммите можно воспользоваться библиотекой lint-staged . Она позволяет осуществлять линтинг только тех файлов, которые изменились
| } | ||
|
|
||
| export class HTTPTransport { | ||
| get: HTTPMethod = (url: string, options: Options) => { |
There was a problem hiding this comment.
(url: string, options: Options) не нужно типизировать, так как HTTPMethod раздает типы
get: HTTPMethod = (url, options) => {
No description provided.