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

<WRAP left round box 80%>
===== Общее о PR =====
Обязательным условием для сдачи ЛР является корректный PR, созданный в репозитории группы по дисциплине.
  * Исходный код представляет из себя задание к ЛР, //выполненное студентом самостоятельно// и принятое платформой e.moevm.info в элементе курса, соответствующем ЛР 
      * Задание на ЛР считается выполненным только в случае успешного прохождения всех тестов
  * Исходный код должен быть загружен в репозиторий группы по дисциплине
      * [[courses:github_rules]]
  * Создан ([[https://docs.github.com/ru/desktop/contributing-and-collaborating-using-github-desktop/working-with-your-remote-repository-on-github-or-github-enterprise/creating-an-issue-or-pull-request|PR]]) на github-странице репозитория

<WRAP left round important 80%>
**Примечание**: если обнаруживается, что студент выполнил лабораторную работу не самостоятельно, то PR с заимствованной ЛР закрывается, а студенту необходимо решить другой вариант лабораторной работы
</WRAP>


===== Корректный PR =====
Загруженный PR считается **корректным** только в случае прохождения **всех автоматических проверок** (PR имеет лейбл **passed**).
<WRAP left round info 80%>
  * Подробнее об автоматической проверке и лейблах PR: [[courses:github_rules#автоматическая_проверка_пуллреквестов_лэйблы|Автоматическая проверка пуллреквестов. Лэйблы]]
  * **Корректный PR** является **допуском к защите** ЛР и **проверяется преподавателем**
      * Этап защиты будет описан позднее
      * Описание проверки PR преподавателем приведено далее
  * Другие PR (закрытые / не прошедшие проверку / с метками Failed или Violated /  пр.) не позволяют студенту защищать ЛР и получать баллы за PR.
      * Чтобы преподаватель проверил PR, нужно исправить все замечания проверяющей системы, добиться метки Passed. 
 
</WRAP>


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

==== Преподавателем проверяется ====
* **Исходный код**
      * Форматировние и стиль кода
      * Логика и алгоритм действий программы
<WRAP left round tip 80%>
Если преподаватель не проверил ваш PR (новый или исправленный), нужно подождать некоторое время или обратиться к соответствующему разделу: [[pr_rules#Трудности коммуникации|Трудности коммуникации]]
</WRAP>


==== Студентом исправляется ====
В случае, если в исходный код необходимо внести изменения, преподаватель **оставляет комментарий** с их описанием и **закрывает PR**.
  * Замечания оставленные преподавателем **являются обязательными для исправления**
  * Комментарии преподавателя **выполняются в том же PR**, путём внесения изменений в существующую ветку и переоткрытия PR
      * Подробнее: [[courses:github_rules#что_делать_если_мой_pull_request_был_закрыт|Что делать, если мой Pull Request был закрыт?]]

<WRAP left round important 80%>
    * Все исправления **должны быть внесены в течение 1 месяца** после проверки преподавателя (но до глобального дедлайна дисциплины)
        * Т.е. если ваш PR был проверен 17.10, изменения должны быть внесены до 17.11 включительно!
        * В случае опоздания оценка за PR снижается на 1 балл
    * **Количество проверок преподавателем** одной ЛР **ограничено 4 проверками**
        * Cтарайтесь внимательно следовать оставленным комментариям!
        * В случае исчерпания количества проверок и не сдаче PR - PR более не проверяется, а баллы за него не учитываются.
</WRAP>

===== Учёт PR в оценивании ЛР =====
<WRAP left round important 80%>
Лабораторные работы считаются зачтёнными (т.е. вы получили данные баллы в рейтинг) **только после мержа соответствующего pull-request’а преподавателем!**

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

===== Трудности коммуникации =====
<WRAP left round box 80%>
==== Общее ====
Если преподаватель по лабораторным работам длительное время (больше 2 недель) не проверяет ваш pull-request (новый или исправленный), без предварительной договоренностей о времени проверок, то необходимо написать ему письмо на почту и уточнить, когда преподаватель проверить ваши pull-request’ы.

<WRAP left round important 80%>
Ответственными за вашу работу являетесь вы, и если вы не интересуетесь временем проверок ваших работ, то ответственность за последствия лежит на вас.
</WRAP>

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

<WRAP left round tip 80%>
Если ответа нет больше недели с момента повторной отправки и вы уверены в правильности оформления письма, то добавьте //в копию письма куратора дисциплины//: __Иванова Дмитрия Владимировича__	
</WRAP>
</WRAP>