ES
я читал про гномов
Size: a a a
2017 September 12
2017 September 13
ES
после тренировки что-то я устал 🙁
IG
@terrakok то есть мне нужно просто разделить мердж реквест на 5 мердж реквестов? По одному на подпункт в issue, так?
IG
Каждый последующий будет включать в себя изменения из предыдущего. Это же норм?
IG
И вопрос - зачем такое разделение необходимо?
Это вроде не такая крупная фича, всего 22 файла затронуто.
Это вроде не такая крупная фича, всего 22 файла затронуто.
KT
Это очень крупная фича
KT
Дело не в количестве файлов
KT
А в отдельных фичах
IG
@terrakok итого - разные MR на каждый пункт в issue? И норм если каждый последующий будет содержать изменения из предыдущего, так?
KT
Пока я не принял первый, второй не надо создавать
IG
Чувствую это надолго затянется, в таком случае. Ну ладно, разделю вечером.
IG
@terrakok может лучше тегами помечать MR в таком случае? Или WIP вешать.
Мы у себя завели теги специальные для таких случаев.
Мы у себя завели теги специальные для таких случаев.
KT
За это не беспокойтесь. Так как когда ваш первый мр будет принят, я его солью не в девелоп, а ветку вашей фичи, и все остальные мр по этой фиче тоже будут идти туда. Когда фича будет готова, то будет общий мр в девелоп. Я ещё раз все посмотрю и солью
IG
Не, меня сейчас больше всего беспокоит следующее: вы в Питере, я в Новосибирске. Когда вы смотрите MR я уже сплю, скорее всего и оперативно открыть следующий MR не смогу. Соответственно, чтобы замерджить целую фичу придётся потратить почти неделю)
IG
При том что на саму фичу ушло от силы 2-3 часа)
KT
Не беспокойтесь. Хуже торопиться, а потом разгребать унылый код.
И это происходит, потому что вы для меня пока не более знакомой Аватарки
И это происходит, потому что вы для меня пока не более знакомой Аватарки
KT
Если действительно ваши мр будут соответствовать качеству и принятому походу, то далее все будет быстрее
KT
И несколько фич будут объединяться
KT
А мне не придется подробно вчитваться в код и сравнивать его с документацией
IG
Окич