Содержание
Правила сдачи лабораторных работ. 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’ы и не отвечает на письмо больше недели, то необходимо сделать следующее:
- Проверить правильность оформления своего письма и указанный в письме адрес преподавателя (любые письма, не соответствующие требованиям оформления, игнорируются)
- Отправить ваше письмо повторно, так как оно могло потеряться на почте преподавателя
Если ответа нет больше недели с момента повторной отправки и вы уверены в правильности оформления письма, то добавьте в копию письма куратора дисциплины: Иванова Дмитрия Владимировича