отправил на pull request, получил комментарий мол убрать файлы в которых не произошли изменения (в некоторых файлах не изменилась логика, а пробел где-то добавлен, ну бывает не только пробел, а например более читабельный код стал, но сама логика не изменилась). В общем-то, таким образом получилось что нужно больше 500+ файлов редачить. Говорит что мол таким образом в annotate будет проще разобраться. Что делать в таких ситуациях? Заранее напишу, что процесс ранее не был обговорен. Только решили его внедрять, а изменений уже было много.
Эм, говорить что ты рефакторил код стайл и это тоже часть изменений и пусть не ленится и посмотрит всё
Вообще по хорошему фичи не стоит смешивать с рефакторингом. Если так хочется отрефачить, то лучше сделать это отдельной веткой, иначе не найдете потом что, когда, где и зачем изменялось
Можно откатить те коммиты, в которых кроме рефактора изменений не было. Вообще, лучше не трогать файлы просто для рефактора в стиле пробелы поставить. Банально, могут быть мерж конфликты. Про читать код ревьюверу 500+ файлов промолчу... И не делать пулл реквесты только после такого большого кол-ва изменений. Ну это на будущее А так, скажи, что это дохера и спроси у него че делать. Или сиди редач)
Спасибо за подробный ответ) Похоже из-за своих же глупостей и буду страдать
Обсуждают сегодня