Skip to content

fix documentation#167

Open
rifcrat wants to merge 1 commit intomainfrom
fix_documentation
Open

fix documentation#167
rifcrat wants to merge 1 commit intomainfrom
fix_documentation

Conversation

@rifcrat
Copy link
Copy Markdown

@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

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%65, 83, 132–133, 155–164, 176, 240–241, 243–244, 252–257, 266, 273–275, 299, 316, 346–357, 375
   exc_handlers.py32391%36, 43, 50
   lecturer.py109992%70–71, 81, 87–88, 220, 228, 248, 254
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 🔥 25.784s ⏱️

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

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

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