Skip to content

Hw2 mirny#7

Open
head-ease08 wants to merge 14 commits intoPython-BI-2025-26:mainfrom
head-ease08:HW2_Mirny
Open

Hw2 mirny#7
head-ease08 wants to merge 14 commits intoPython-BI-2025-26:mainfrom
head-ease08:HW2_Mirny

Conversation

@head-ease08
Copy link

Add calculator.py, README.md, include team_call.png

head-ease08 and others added 14 commits September 17, 2025 12:27
Added main() that runs calculator logic.
Implemented div() function that takes two numbers and divides
the first by the second. Handles division by zero.
HW2 main and division functions
Sergey added multiplication function named mul(a, b)
HW2_Mirny: add subtraction function (sub) and README.md
Add multiplication function
Revert "Added multiplication function"
Sergey added multiplication function named mul(a, b)
Added multiplication function
@Valeriisht
Copy link

Valeriisht commented Sep 24, 2025

Привет!✨

Работа хорошая! Замечание насчет коммитов - пишем в настоящем времени: Add, а не Added и с большой буквы. Также есть небольшие замечания к коду - помечу в комментариях. Хвалю, что для фото созвона сделали отдельную папку!

Вопросик с форком репозитория: SergeyIlin322 сделал форк оригинального репозитория Никиты, а не тимлида, как написано в условии. Я сниму за это баллы только Сергею, поэтому балл у участников команды будет различаться.

Итоговые баллы за задание: 10/10 (README) + 10/10 (Forks) + 80/80 (код и коммиты) + 10 доп. за картинку созвона = 100/100 + 10 доп.

  • У Сергея - 2.5 балла за форк = 97.5/100 + 10 доп.

Умнички, хорошо поработали! Удачи дальше!)

(тимлид, не забудь показать ревью остальным участникам команды)

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.

5 participants