Skip to content

Conversation

@RamanMukhin
Copy link
Owner

@RamanMukhin RamanMukhin commented Jul 8, 2021

NestJS

  1. ссылка на задание!
  2. Дата сдачи: 08.07.2021 / Дата дедлайна 11.07.2021
  3. Самопроверка: 100 баллов

Выполненные пункты:

  1. Guards должны использоваться для работы с авторизацией/аутентификацией: ** Выполнено.**
  2. В приложении должны использоваться модули для разбиения структуры приложения на различные части (User, Board, Task): ** Выполнено.**
  3. Exception filters должны использоваться для обработки исключений в приложении: ** Выполнено.**
  4. @nestjs/typeorm должен использоваться для работы с базой данных: ** Выполнено.**
  5. Для логирования может использоваться встроенный Logger или кастомная имплементация: ** Выполнено.**
  6. Все внешние зависимости для модулей/классов должны предоставляться с помощью механизма dependency injection: ** Выполнено**
  7. В зависисимости от env переменной USE_FASTIFY Nest.js должен использовать или express или fastify: ** Выполнено**
  8. Необходимо сравнить производительность Nest.js с использованием express и fastify (можно использовать для этих целей artillery): ** Выполнено**

Штрафы:

  • Наличие изменений в тестах либо в workflow: Отсутствует.
  • Внесение изменений в репозиторий после дедлайна не считая коммиты, вносящие изменения только в Readme.md и другую документацию): Отсутствует.
  • За отсутствие отдельной ветки для разработки: Отсутствует.
  • За отсутствие Pull Request: Отсутствует.
  • За неполную информацию в описании Pull Request (отсутствует либо некорректен один из 3 обязательных пунктов): Отсутствует.
  • За каждую ошибку линтера при запуске npm run lint на основе локального конфига -5 баллов (именно errors, не warnings): Отсутствует.
  • За каждый непроходящий тест npm run test:auth: Отсутствует.
  • Меньше 3 коммитов (не считая коммиты, вносящие изменения только в Readme.md и другую документацию) : Отсутствует.

#ИТОГО:

30 + 30 + 30 + 30 + 30 + 30 + 30 + 30 0 = 240 баллов

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