В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

JUG.ru

1489 membersпожаловаться на группу
2022 January 27

С

Сурен in JUG.ru
#question Как быть если ты решаешь тикет, и видишь какие-то неточности, добавил где-то отступы, добавил в нескольких классах javaDoc, отрефакторил метод и таких совсем мелких улучшений слишком много. И как быть каждый раз это все описывать? А смысл если это будет видно в коммите? Но если так не делать то все изменения сольются в 1 одном коммите. Как быть с этой дилеммой
источник
20:25пожаловаться#1

IS

Ilya Sazonov in JUG.ru
Мне кажется по хорошему надо сплитить коммит и делать несколько сообщений.
источник
20:28пожаловаться#2

C

Chatbro in JUG.ru
Jurij Zigmund
исповедь
источник
20:30пожаловаться#3

С

Сурен in JUG.ru
Ну если у изменилось классов не 2-3 а 20-23 это каждый раз выборочно потом изменения выискивать которые должны войти в следующий коммит? И ты занимаешься ресерчим их, пытаешься засташить измнения которые не должны войти. А если что забудешь? И это не 5 минут времени займет выискивать что где-то ты джавадок поправил или в голове держать что надо поправить
источник
20:30пожаловаться#4

C

Chatbro in JUG.ru
Александр М
Мы стараемся сначала делать форматирование отдельным коммитом, а потом уже требуемое изменение.
источник
20:31пожаловаться#5

C

Chatbro in JUG.ru
Андрей Бурмагин
А мне не нравится такой подход. Если не уверен - не коммить. Действительно ли это хорошее решение должны контроллировать ревьюеры.
источник
20:31пожаловаться#6

IK

Ivan Kostrov in JUG.ru
В IDEA кстати можно не только файлы, но и конкретные изменения включать/исключать в коммит. Если проект хороший и "сроки не горят" то время не проблема, это сохранит время в будущем. А если проект плохой - то все равно все умрет ))
источник
20:31пожаловаться#7

IS

Ilya Sazonov in JUG.ru
Стандартная рекомендация - коммитить часто часто. Правда не часто я видел, чтобы люди на самом деле так делали
источник
20:32пожаловаться#8

С

Сурен in JUG.ru
Это в идея шелве? Пользуюсь гит сташ и там выборочно в файле не получается и иногда страдаешь. Надо будет распробовать
источник
20:33пожаловаться#9

C

Chatbro in JUG.ru
Jurij Zigmund
мне кажется в диффе видно когда форматили код и вопросов "почему так отформатили?" не должно возникать :)))
источник
20:33пожаловаться#10

C

Chatbro in JUG.ru
Андрей Бурмагин
Между сквошем и антисквошем нужно найти баланс.
источник
20:33пожаловаться#11

IK

Ivan Kostrov in JUG.ru
В diff окне коммита в IDEA
источник
20:33пожаловаться#12

C

Chatbro in JUG.ru
Андрей Бурмагин
А в идеале один маленький коммит на маленький пулреквест.
источник
20:34пожаловаться#13

S

StShadow in JUG.ru
ой был у нас один такой
мешал рефакторинг с бизнес-логикой
ему на ревью надавали
обиделся
но потом уже было лучше
источник
20:34пожаловаться#14

C

Chatbro in JUG.ru
Игорь Иванов
иногда даже ревьювер может толком не понимать, хорошо это или плохо. Буквально в прошлом месяце такое было, наркоманский фикс для наркоманской проблемы в древнем коде. Я писал, трое неглупых человек ревьювили, ни у кого не было чёткой уверенности, кошерный ли это метод починки, или можно лучше(
источник
20:35пожаловаться#15

IS

Ilya Sazonov in JUG.ru
Вообще сказали ему, чтобы в ветках, отрезанных для бизнес задач ничего не писал, что к бизнес задаче отношения не имеет?
источник
20:36пожаловаться#16

C

Chatbro in JUG.ru
Андрей Бурмагин
В таком случае я согласен с автором, лучше предложить альтернативные варианты и объяснить, почему выбрал текущий, а не ныть, что "это всего лишь устранение симптомов".
источник
20:37пожаловаться#17

S

StShadow in JUG.ru
что-то вроде этого
мол хочешь порефакторить - ок, но в отдельном коммите, где только рефакторинг, в отдельной ветке и с понятным коммитом, что только безопасный рефакторинг
источник
20:37пожаловаться#18

IS

Ilya Sazonov in JUG.ru
И получается ответ на первоначальный вопрос - не делайте так вообще?
источник
20:37пожаловаться#19

IK

Ivan Kostrov in JUG.ru
Т.е. говнокод намеренно не исправлять при изменении бизнес-логики, чтобы не было рефакторинга. А юнит тесты не дают уверенности в безопасности рефакторинга?
источник
20:38пожаловаться#20