Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

HW1 is completed #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

HW1 is completed #1

wants to merge 2 commits into from

Conversation

timersha
Copy link
Owner

@timersha timersha commented Oct 14, 2024 •
edited
Loading

Чек-лист студента

  • Я перечитал задание после решения.
  • Я запустил go mod tidy.
  • Я удалил .sync файл.
  • Я использовал stringutil.Reverse.
  • Я сравнил объем кода с объемом авторского решения.
    • main.go ~12 строк

Критерии оценки

  • Пайплайн зелёный - 4 балла
  • Используется stringutil - 4 балла
  • Понятность и чистота кода - до 2 баллов

agneum
agneum previously approved these changes Oct 16, 2024
View reviewed changes
Copy link
Collaborator

@agneum agneum left a comment

Choose a reason for hiding this comment

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

Спасибо за выполненное задание. Хорошая работа
Итого: 10 баллов из 10. Принято

Прошу прислать инвайт в репозиторий, чтобы аппрув имел вес, и проверить настройки мержа в репозитории. Похоже, что сейчас мерж в мастер открыт без аппрува.

Желаю успехов в прохождении курса!

hw01_hello_otus/go.mod Outdated
@@ -1,3 +1,5 @@
module github.com/fixme_my_friend/hw01_hello_otus
module github.com/timersha/hw01_hello_otus
Copy link
Collaborator

Choose a reason for hiding this comment

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

Следует указать полный путь, включающий название репозитория

Copy link
Owner Author

Choose a reason for hiding this comment

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

этот путь? https://github.com/timersha/golang-tests/tree/hw01_hello_otus/hw01_hello_otus

Copy link
Collaborator

Choose a reason for hiding this comment

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

github.com/timersha/golang-tests/hw01_hello_otus

@timersha
Copy link
Owner Author

Спасибо за выполненное задание. Хорошая работа Итого: 10 баллов из 10. Принято

Прошу прислать инвайт в репозиторий, чтобы аппрув имел вес, и проверить настройки мержа в репозитории. Похоже, что сейчас мерж в мастер открыт без аппрува.

Желаю успехов в прохождении курса!

пригласил в репозиторий, правило добавил, спасибо

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agneum agneum agneum left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timersha @agneum