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

Question

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

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

#android #mobile #programming #russian

0

25.08.2021

4 ответов

23 просмотра

Nikita Kladov

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

0

25.08.2021

ℕo ℕame

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

0

25.08.2021

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

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

0

25.08.2021

Андрей · Accepted Answer

Андрей

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

0

25.08.2021

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

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

4 ответов

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