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

групп:

каналов:

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

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

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

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

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

go.backupland.com

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

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

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

Size: a a a

Frontend UA

615 membersпожаловаться на группу
2020 October 15

Вт

Ві тя in Frontend UA
Dmytro Dovhan
категорично )

с чего бы они должны быть атомарные?
Что б ревью проще делать, что б тикет подвязать, что б если что сломалось ревертнешь одну фичу а не пять, и еще много других причин. Это уже стандарт разработки с гитом
источник
15:26пожаловаться#1

Вт

Ві тя in Frontend UA
Stas G
в конце рабочего дня можно сделать wip коммит, запушить его, а на следующий день закончить и сделать форс-пуш
я б запретил форспуш)

Можно конечно засташить, но зачем. Вдруг завтра на другой тачке будешь код писать
источник
15:27пожаловаться#2

SG

Stas G in Frontend UA
ну и что, что на другой?
источник
15:27пожаловаться#3

SG

Stas G in Frontend UA
спулил себе ветку и работаешь в ней дальше
источник
15:27пожаловаться#4

SG

Stas G in Frontend UA
запрещать форс-пуш можно разве что в мастер
источник
15:28пожаловаться#5

SG

Stas G in Frontend UA
иначе просто кто-то не умеет с гитом работать)
источник
15:28пожаловаться#6

EO

Eugene Obrezkov in Frontend UA
Stas G
тут кажется никто не слышал про такой флаг как --amend
и делать форс пуш в ветку, которую всё равно сквошить будешь?
источник
15:28пожаловаться#7

SG

Stas G in Frontend UA
ну я допустим свои ветки не сквошу
источник
15:28пожаловаться#8

SG

Stas G in Frontend UA
я просто навожу порядок в коммитах
источник
15:28пожаловаться#9

EO

Eugene Obrezkov in Frontend UA
ну тогда это кейс с ребейсом
источник
15:28пожаловаться#10

DD

Dmytro Dovhan in Frontend UA
Ві тя
Что б ревью проще делать, что б тикет подвязать, что б если что сломалось ревертнешь одну фичу а не пять, и еще много других причин. Это уже стандарт разработки с гитом
первый раз слышу про стандарт,

про ревью тоже вкусовщина,

лично мне удобнее атомарными комитами ревьювить - можно проследить порядок изменений

чем один пр на 30 измененых фалов
источник
15:28пожаловаться#11

EO

Eugene Obrezkov in Frontend UA
ты себе хулиганишь и потом ребейсишь свою ветку, чтобы история была норм с последующим мержем с ff
источник
15:29пожаловаться#12

EO

Eugene Obrezkov in Frontend UA
это тоже валидный flow
источник
15:29пожаловаться#13

EO

Eugene Obrezkov in Frontend UA
Dmytro Dovhan
первый раз слышу про стандарт,

про ревью тоже вкусовщина,

лично мне удобнее атомарными комитами ревьювить - можно проследить порядок изменений

чем один пр на 30 измененых фалов
иногда ты не можешь заливать атомарно изменения
источник
15:30пожаловаться#14

Вт

Ві тя in Frontend UA
Dmytro Dovhan
первый раз слышу про стандарт,

про ревью тоже вкусовщина,

лично мне удобнее атомарными комитами ревьювить - можно проследить порядок изменений

чем один пр на 30 измененых фалов
https://medium.com/@fagnerbrack/one-pull-request-one-concern-e84a27dfe9f1
Medium
One Pull Request. One Concern.
The importance of a Pull Request lies on the definition of an atomic concern
источник
15:30пожаловаться#15

Вт

Ві тя in Frontend UA
Вот хорошо обьясняется что имею ввиду
источник
15:31пожаловаться#16

Вт

Ві тя in Frontend UA
Суть в том что в пр не должно быть изменений рефакторингов фиксов чего либо, не касающегося сути пр. Если пр «сделать харашо» то надо дробить задачу. Соответственно, и полезности многих коммитов при мерже не много. Приходится Наоборот, надо следить что б каждый коммит не сломал СІ.
источник
15:32пожаловаться#17

DD

Dmytro Dovhan in Frontend UA
с одной стороны валидно, с другой пр на каждую правку опечатки? это что бы датацетры крутящие ci не простаивали?

еще из плюсов атомарных комитов, в случае нахождения какой либо проблемы через git bisect можно легко найти конкретный атомарный комит который поломал, это же явно проще чем искать что именно в сквошнутом комите приводит к проблемам?
источник
15:37пожаловаться#18

AR

Alexey Raspopov in Frontend UA
Eugene Obrezkov
will it work on CI?
fuck, I forgot to include the module
what is happening?
arghhh
fix lint
fix lint
fix lint
fix lint
источник
15:37пожаловаться#19

EO

Eugene Obrezkov in Frontend UA
Dmytro Dovhan
с одной стороны валидно, с другой пр на каждую правку опечатки? это что бы датацетры крутящие ci не простаивали?

еще из плюсов атомарных комитов, в случае нахождения какой либо проблемы через git bisect можно легко найти конкретный атомарный комит который поломал, это же явно проще чем искать что именно в сквошнутом комите приводит к проблемам?
так после сквошей фичи в мастер попадают с нормальной историей и бисект будет себе работать
источник
15:39пожаловаться#20