Skip to content

feat: Complete PrompTab project implementation with backend, frontend, docs and CI/CD#1

Merged
natigaminov merged 3 commits intomainfrom
dev
Jul 15, 2025
Merged

feat: Complete PrompTab project implementation with backend, frontend, docs and CI/CD#1
natigaminov merged 3 commits intomainfrom
dev

Conversation

@natigaminov
Copy link
Copy Markdown
Contributor

Complete PrompTab project implementation with backend, frontend, docs and CI/CD

@Vl-Tershch Vl-Tershch added the enhancement New feature or request label Jul 14, 2025
@Vl-Tershch Vl-Tershch self-requested a review July 14, 2025 13:18
Copy link
Copy Markdown

@Vl-Tershch Vl-Tershch left a comment

Choose a reason for hiding this comment

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

Поскольку начальная реализация проекта, замечания скорее стилистические -

  1. Самописные исключения я бы для удобства разнес по отдельным файлам для удобства расширения.
  2. backend/scripts/run_tests.sh кажется не сильно нужным, поскольку можно просто запускать тесты на этапе CI.
  3. Все модели/датаклассы я бы тоже разносил по отдельным файлам для удобства внесения изменений.
  4. Разделы в README.md, а именно CI/CD Pipeline, Security кажутся несколько избыточными.

В целом, считаю, что замечания носят скорее информативный характер и следует обратить на них внимание при дальнейшей разработке.

@natigaminov natigaminov merged commit 302219d into main Jul 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants