Ks 1 2 #171
Ks 1 2 #171
Conversation
src/main/kotlin/Lessson_1/L1.kt
Outdated
| @@ -0,0 +1,6 @@ | |||
| package Lessson_1 | |||
|
|
|||
| fun main() { | |||
There was a problem hiding this comment.
Есть нарушения в git флоу, то есть в логике работы с коммитами и ветками.
Обрати внимание, сначала ты сделал одну задачу, запушил и все ок.
Затем ты закоммитил в эту же ветку другую задачу (либо создал следующую ветку из предыдущей рабочей ветки) – это видно в истории коммитов.
Что из этого следует? В пулл реквест с одной задачей попала задача, которая должна быть в другой ветке. Это нарушает суть разделения кода задач по веткам. Задача из одной ветки не должна знать и пересекаться с задачей из другой ветки.
На большом проекте затрагивается много файлов и если в одном ПР делать другие изменения (из другой задачи), то при слиянии может быть много проблем и/или гит конфликтов. Например, над кодом другой задачи уже работает программист, изменения из второй задачи не проверены и не протестированы (тестировщики проверяют функционал по конкретному тех. заданию в рамках данного описания задачи в конкретной ветке).
Новые ветки создаем только из мастера (который предварительно надо обновить, если до этого мержились задачи в гитхабе), чтобы не было пересечений с другими тасками)
Что делать?
в данном пулл реквесте надо:
- переключиться в ветку текущего ПР
- удалить лишний файл с задачей (физически из проекта)
- сделать коммит (с изменением удаления)
- прислать ссылку на этот же пулл реквест снова в бот
И новые ветки всегда создаем из основной ветки master.
src/main/kotlin/Lessson_1/L2.kt
Outdated
| var workers = 2000 | ||
| println(workers) | ||
| */ | ||
| var workers = 1999 |
There was a problem hiding this comment.
Здесь переменная объявляется заново. По условию задачи нужно закомментировать только вывод 2000, а значение переменной перезаписать новым.
src/main/kotlin/Lessson_1/L2.kt
Outdated
|
|
||
|
|
||
| fun main() { | ||
| var ordersOnWeb = 75 |
src/main/kotlin/Lessson_1/L2.kt
Outdated
|
|
||
|
|
||
| fun main() { | ||
| var ordersOnWeb = 75 |
.idea/vcs.xml
Outdated
There was a problem hiding this comment.
Нужно добавить эти файлы .idea в gitignor, чтобы они не коммитились
No description provided.