169 похожих чатов

Привет, такой вопрос. Занимался разработкой апки, собралось много коммитов, когда

отправил на pull request, получил комментарий мол убрать файлы в которых не произошли изменения (в некоторых файлах не изменилась логика, а пробел где-то добавлен, ну бывает не только пробел, а например более читабельный код стал, но сама логика не изменилась). В общем-то, таким образом получилось что нужно больше 500+ файлов редачить. Говорит что мол таким образом в annotate будет проще разобраться. Что делать в таких ситуациях? Заранее напишу, что процесс ранее не был обговорен. Только решили его внедрять, а изменений уже было много.

4 ответов

21 просмотр

Эм, говорить что ты рефакторил код стайл и это тоже часть изменений и пусть не ленится и посмотрит всё

Вообще по хорошему фичи не стоит смешивать с рефакторингом. Если так хочется отрефачить, то лучше сделать это отдельной веткой, иначе не найдете потом что, когда, где и зачем изменялось

Можно откатить те коммиты, в которых кроме рефактора изменений не было. Вообще, лучше не трогать файлы просто для рефактора в стиле пробелы поставить. Банально, могут быть мерж конфликты. Про читать код ревьюверу 500+ файлов промолчу... И не делать пулл реквесты только после такого большого кол-ва изменений. Ну это на будущее А так, скажи, что это дохера и спроси у него че делать. Или сиди редач)

Вадим 👨‍💻- Автор вопроса

Спасибо за подробный ответ) Похоже из-за своих же глупостей и буду страдать

Похожие вопросы

Обсуждают сегодня

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Я вот подумал. SSE выполняет операции максимум с 64-битной точностью. А FPU - всегда с 80-битной. Разве не должно быть FPU точнее тогда?
The Bird of Hermes
13
как быть с принтером? такой подход прокатит?
zamtmn
12
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
Я не понимаю, это троллинг или что? Швабрика поддерживают, который буквально пишет на ассемблере взаимодействия с винапи. Я это ещё написал загрузчик и хоть что-то изучаю в о...
Shadow Akira
6
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Привет. Подскажите, как правильно сматчить лист фиксированного размера, чтобы компилятор не говорил мне о неполном паттерне? Допустим что-то такое [x', y'] = sort [x, y]?
Arseny
8
Карта сайта