Skip to content

Hw2_Vashunina#10

Open
dashvash wants to merge 23 commits intoPython-BI-2024-25:mainfrom
dashvash:HW2_Vashunina
Open

Hw2_Vashunina#10
dashvash wants to merge 23 commits intoPython-BI-2024-25:mainfrom
dashvash:HW2_Vashunina

Conversation

@dashvash
Copy link

Our first attemt

@ISonets
Copy link

ISonets commented Sep 26, 2024

привет!
форки от сокомандников вижу, все ок.
коммиты вижу, но хотелось бы видеть их более подробно, не просто update README.md, а что конкретно было обновлено, например Update "usage' part of README.
к слову, порадовал подробный ридми!
с кодом все ок, но отступы у функций неправильные (3 пробела вместо 4. на работоспособность не влияет, но это не соответствует pep8).
итого:
10 баллов за доп.задание
10 баллов за ридми
10 баллов за форки
за 4 арифметических действия: 16 баллов*4 =64 баллов. сделано красиво, ясно.
за main:написан верно,16 баллов. в будущем останется только добавить вызов main().

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.

4 participants