Size: a a a

🎄.NET Talks: Evergreen🎄

2020 June 15

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Я Акула Туруруру
Вот сейчас подумал, что, наверное, при нормальной культуре разработки ничего другого и не требуется. У нас сейчас просто люди не пользуются пулл-реквестами, мы до этого сидели на каком-то всратом гит сервере, где их и не было, скорее всего. Теперь на гитхаб перевезли проект, но по-прежнему мержим develop в staging вместо пулл-реквестов =/ Надо переставать
Не, ну можно и автотесты для каждого коммита сделать
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Вроде те же gh actions умеют на каждый коммит возбуждаться
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Просто это не так удобно, потому что ты получаешь ответ постфактум
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Уже после того, как вмержился
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
И, разумеется, вместо того, чтобы чинить упавшие тесты, ты уже побежал пилить фичи
источник

KF

Kirill Fimchenko in 🎄.NET Talks: Evergreen🎄
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
И сегодня тебе лень переключаться на починку тестов — ведь это угроза твоей работоспособности, угроза бизнесу! И завтра лень. А послезавтра срочный аврал, и ты не смог заняться
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Ну а послепослезавтра уже суббота. В понедельник пишешь посонам — мол, тест уже неделю красный, ну его нафиг, давайте замьютим
источник

ЯТ

Я Акула Туруруру... in 🎄.NET Talks: Evergreen🎄
Dr. Friedrich von Never
Ну а послепослезавтра уже суббота. В понедельник пишешь посонам — мол, тест уже неделю красный, ну его нафиг, давайте замьютим
Так и происходит. Один челик вообще мержит изменения, с которыми солюшен не билдится из-за сломанных тестов. Вот хочется как-то это пресечь, чтобы физически невозможно было смержить такое
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Я Акула Туруруру
Так и происходит. Один челик вообще мержит изменения, с которыми солюшен не билдится из-за сломанных тестов. Вот хочется как-то это пресечь, чтобы физически невозможно было смержить такое
Я не сомневаюсь, что так происходит. Ты думаешь, что я шутил, а я совсем не шутил, видал такое уже тыщу раз :)
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Dr. Friedrich von Never
Я не сомневаюсь, что так происходит. Ты думаешь, что я шутил, а я совсем не шутил, видал такое уже тыщу раз :)
(вы думаете, что я кого-то шеймлю, а я никого не шеймлю: на мне самом парочка таких тестов висит, и я отнекиваюсь вот этими самыми отговорками)
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
В общем, да, внедрение обязательных пул-реквестов с зелёными тестами — это отличная идея.
источник

ЯТ

Я Акула Туруруру... in 🎄.NET Talks: Evergreen🎄
Dr. Friedrich von Never
В общем, да, внедрение обязательных пул-реквестов с зелёными тестами — это отличная идея.
У нас сейчас такая схема - делаем фичи в отдельных ветках, мержим их в develop (вот тут стоит заменить на пулл-реквесты), потом мержим develop в staging, после тестирования на staging мержим его в мастер. 2 последних этапа стоит оставить как есть, обычными мержами?
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Я Акула Туруруру
У нас сейчас такая схема - делаем фичи в отдельных ветках, мержим их в develop (вот тут стоит заменить на пулл-реквесты), потом мержим develop в staging, после тестирования на staging мержим его в мастер. 2 последних этапа стоит оставить как есть, обычными мержами?
Ну, у меня какое-то время была схема, когда develop мержился в master/staging через fast-forward.
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
А потом для основных проектов я просто забил на остальные бранчи, работаю через PR, мержу в мастер, а релизы помечаю тегами :)
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Но в твоей схеме, если она тебя устраивает, я не вижу проблем.
источник

ЯТ

Я Акула Туруруру... in 🎄.NET Talks: Evergreen🎄
Ок, спасибо. Предложу этот вариант, поглядим
источник

R

RondoK in 🎄.NET Talks: Evergreen🎄
источник

Т8

Т-34 85 in 🎄.NET Talks: Evergreen🎄
Dr. Friedrich von Never
Ну а послепослезавтра уже суббота. В понедельник пишешь посонам — мол, тест уже неделю красный, ну его нафиг, давайте замьютим
А чо он красный, надо его поправить, или баг в проекте?
источник

Dv

Dr. Friedrich von Ne... in 🎄.NET Talks: Evergreen🎄
Т-34 85
А чо он красный, надо его поправить, или баг в проекте?
А кто знает? Намержили чота, потом намержили сверху, и затем пришли и ещё чуток намержили. В середине сломали компиляцию, а починили только на следующий день, и вот после этого увидели, что всё сломано %)
источник