Содержание

Правила сдачи лабораторных работ. PullRequest

Общее о PR

Обязательным условием для сдачи ЛР является корректный PR, созданный в репозитории группы по дисциплине.

  • Исходный код представляет из себя задание к ЛР, выполненное студентом самостоятельно и принятое платформой e.moevm.info в элементе курса, соответствующем ЛР
    • Задание на ЛР считается выполненным только в случае успешного прохождения всех тестов
  • Исходный код должен быть загружен в репозиторий группы по дисциплине
  • Создан (PR) на github-странице репозитория

Примечание: если обнаруживается, что студент выполнил лабораторную работу не самостоятельно, то PR с заимствованной ЛР закрывается, а студенту необходимо решить другой вариант лабораторной работы

Корректный PR

Загруженный PR считается корректным только в случае прохождения всех автоматических проверок (PR имеет лейбл passed).

  • Подробнее об автоматической проверке и лейблах PR: Автоматическая проверка пуллреквестов. Лэйблы
  • Корректный PR является допуском к защите ЛР и проверяется преподавателем
    • Этап защиты будет описан позднее
    • Описание проверки PR преподавателем приведено далее
  • Другие PR (закрытые / не прошедшие проверку / с метками Failed или Violated / пр.) не позволяют студенту защищать ЛР и получать баллы за PR.
    • Чтобы преподаватель проверил PR, нужно исправить все замечания проверяющей системы, добиться метки Passed.

Проверка PR

Корректный PR проверяется преподавателем для получения студентом баллов за PR с ЛР.

Преподавателем проверяется

* Исходный код

  • Форматировние и стиль кода
  • Логика и алгоритм действий программы

Если преподаватель не проверил ваш PR (новый или исправленный), нужно подождать некоторое время или обратиться к соответствующему разделу: Трудности коммуникации

Студентом исправляется

В случае, если в исходный код необходимо внести изменения, преподаватель оставляет комментарий с их описанием и закрывает PR.

  • Замечания оставленные преподавателем являются обязательными для исправления
  • Комментарии преподавателя выполняются в том же PR, путём внесения изменений в существующую ветку и переоткрытия PR
  • Все исправления должны быть внесены в течение 1 месяца после проверки преподавателя (но до глобального дедлайна дисциплины)
    • Т.е. если ваш PR был проверен 17.10, изменения должны быть внесены до 17.11 включительно!
    • В случае опоздания оценка за PR снижается на 1 балл
  • Количество проверок преподавателем одной ЛР ограничено 4 проверками
    • Cтарайтесь внимательно следовать оставленным комментариям!
    • В случае исчерпания количества проверок и не сдаче PR - PR более не проверяется, а баллы за него не учитываются.

Учёт PR в оценивании ЛР

Лабораторные работы считаются зачтёнными (т.е. вы получили данные баллы в рейтинг) только после мержа соответствующего pull-request’а преподавателем!

В любых иных случаях работа считается невыполненной и вы не получаете своих баллов за неё.

Трудности коммуникации

Общее

Если преподаватель по лабораторным работам длительное время (больше 2 недель) не проверяет ваш pull-request (новый или исправленный), без предварительной договоренностей о времени проверок, то необходимо написать ему письмо на почту и уточнить, когда преподаватель проверить ваши pull-request’ы.

Ответственными за вашу работу являетесь вы, и если вы не интересуетесь временем проверок ваших работ, то ответственность за последствия лежит на вас.

Особые случаи

Если преподаватель без объяснения причин и предварительных договоренностей не проверяет ваши pull-request’ы и не отвечает на письмо больше недели, то необходимо сделать следующее:

  • Проверить правильность оформления своего письма и указанный в письме адрес преподавателя (любые письма, не соответствующие требованиям оформления, игнорируются)
  • Отправить ваше письмо повторно, так как оно могло потеряться на почте преподавателя

Если ответа нет больше недели с момента повторной отправки и вы уверены в правильности оформления письма, то добавьте в копию письма куратора дисциплины: Иванова Дмитрия Владимировича