Skip to content

Update README.md#47

Merged
Rum-and-tea merged 1 commit intomainfrom
Rum-and-tea-patch-1
Nov 23, 2023
Merged

Update README.md#47
Rum-and-tea merged 1 commit intomainfrom
Rum-and-tea-patch-1

Conversation

@Rum-and-tea
Copy link
Copy Markdown
Contributor

@Rum-and-tea Rum-and-tea commented Nov 22, 2023 •
edited
Loading

Idk...

Изменения

Опечаточка (бекэдН->бекэнд)

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@Rum-and-tea
Update README.md
b247937 
Idk... Мне вот это нужно было сделать?
(Я глупый)
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 22, 2023

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
services_backend
   __main__.py440%1–7
   schemas.py8450%6–9
services_backend/models
   base.py13469%19–22
services_backend/routes
   button.py1061388%65, 87–92, 109, 114, 131, 136, 162, 168
   category.py109595%90, 118, 150–151, 192
TOTAL3223091% 

Summary

Tests Skipped Failures Errors Time
29 0 💤 0 ❌ 0 🔥 1.433s ⏱️

@Temmmmmo
Copy link
Copy Markdown
Member

Temmmmmo commented Nov 23, 2023

Все отлично!
Только убери из описания к ПРу пункт про детали реализации полностью, так как тут их нет, что логично)) И убери свой вопрос оттуда, так как не принято там какие-то вопросы задавать. Это, скорее, просто краткое описание проделанной работы, чтобы человек, проводящий ревью, мог понять, что вообще происходит)

@Rum-and-tea
Copy link
Copy Markdown
Contributor Author

Rum-and-tea commented Nov 23, 2023

Да, я исправил всё

Temmmmmo
Temmmmmo approved these changes Nov 23, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@Temmmmmo Temmmmmo left a comment

Choose a reason for hiding this comment

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

Поздравляю с первым аппрувнутым ПРом))
Можешь мерджить.

@Rum-and-tea Rum-and-tea merged commit f69f4ed into main Nov 23, 2023
@Rum-and-tea Rum-and-tea deleted the Rum-and-tea-patch-1 branch November 23, 2023 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Temmmmmo Temmmmmo Temmmmmo approved these changes

Assignees

@Rum-and-tea Rum-and-tea

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Rum-and-tea @Temmmmmo