Skip to content

HW11 is completed - #11

Open
qeeq72 wants to merge 3 commits into
masterfrom
hw11_telnet_client
Open

HW11 is completed#11
qeeq72 wants to merge 3 commits into
masterfrom
hw11_telnet_client

Conversation

@qeeq72

@qeeq72 qeeq72 commented Dec 20, 2021

Copy link
Copy Markdown
Owner

Домашнее задание №11 «Клиент TELNET»

Чек-лист студента (Что это?)

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

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

Зачёт от 7 баллов

@sshaplygin
sshaplygin self-requested a review December 25, 2021 18:08

@sshaplygin sshaplygin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Здравствуйте. К сожалению, не могу оценить выполнение, так существуют критические ошибки.

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

Нотация:
👍 - Классное, не обычное, оригинальное решение

👀 - Стоит обратить внимание. Рекомендация, которая не влияет на успешность выполнения задания

🔥 - Критичное замечание, которое влияет на сдачу задания

Comment thread hw11_telnet_client/main.go Outdated
Comment thread hw11_telnet_client/telnet.go

@sshaplygin sshaplygin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Здравствуйте. Спасибо за выполненное задание. Отличная работа

Итого: 10/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.

2 participants