Skip to content

fix documentation#167

Merged
Zimovchik merged 8 commits intomainfrom
fix_documentation
Apr 22, 2026
Merged

fix documentation#167
Zimovchik merged 8 commits intomainfrom
fix_documentation

Conversation

@rifcrat
Copy link
Copy Markdown
Contributor

@rifcrat rifcrat commented Apr 19, 2026

Изменил документацию

@github-actions
Copy link
Copy Markdown

💩 Code linting failed, use black and isort to fix it.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 19, 2026

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–7
   exceptions.py43784%35–37, 48–50, 58
rating_api/models
   base.py64494%24–27
   db.py1641889%115, 117, 119, 121, 132, 171, 189, 199, 213, 222–224, 257, 271–283
rating_api/routes
   base.py16194%41
   comment.py1513577%75, 93, 142–143, 165–174, 190, 263–264, 266–267, 275–280, 289, 296–298, 328, 351, 391–402, 429
   exc_handlers.py32391%36, 43, 50
   lecturer.py109992%72–73, 83, 89–90, 230, 238, 260, 266
rating_api/schemas
   base.py12467%6–9
   models.py155398%189, 191, 201
rating_api/utils
   mark.py880%1–20
TOTAL7929688% 

Summary

Tests Skipped Failures Errors Time
88 0 💤 0 ❌ 0 🔥 21.431s ⏱️

Copy link
Copy Markdown
Member

@petrCher petrCher left a comment

Choose a reason for hiding this comment

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

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

@petrCher
Copy link
Copy Markdown
Member

А еще надо использовать форматирование black isort(сейчас вылетает ошибка в Python package / linting (pull_request))

@petrCher
Copy link
Copy Markdown
Member

Еще справа сбоку есть панель, где ты ревью запросил у меня. там надо в development добавить задачу, которую выполняешь, чтобы после мерджа пр(пулл реквеста) задача автоматически закрылась

@petrCher
Copy link
Copy Markdown
Member

Также, я очень надеюсь ты смог у себя локально запустить свагер и потыкать там все.
Рекомендую все вопросы по пр писать сюда комментом, а в тг просто писать что откомментил

@github-actions
Copy link
Copy Markdown

💩 Code linting failed, use black and isort to fix it.

@github-actions
Copy link
Copy Markdown

💩 Code linting failed, use black and isort to fix it.

Comment thread rating_api/routes/comment.py
Comment thread rating_api/routes/lecturer.py Outdated
@petrCher petrCher self-requested a review April 21, 2026 08:23
@Zimovchik Zimovchik merged commit a6f6b5a into main Apr 22, 2026
2 checks passed
@Zimovchik Zimovchik deleted the fix_documentation branch April 22, 2026 18:07
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.

переделать документацию ко всем ручкам #146

3 participants