c
portnov
типа 1952 теста из 2017 зелёные — это ок или не ок?
типа 1952 теста из 2017 зелёные — это ок или не ок?
Size: a a a
2021 May 23
Dv
Обычно мы не мержим код, если не замьюченные тесты упали.
c
portnov
понятно же что бывают всякие тесты, которые упали из-за глюка временного какого-нибудь, или тестящие что-то маловажное
понятно же что бывают всякие тесты, которые упали из-за глюка временного какого-нибудь, или тестящие что-то маловажное
Dv
У кого надо есть оверрайд, но чаще всего не мержим
Dv
(у меня нету оверрайда, что примечательно)
c
portnov
а, тесты запускаются до мержа, на PR, как на гитхабе?
а, тесты запускаются до мержа, на PR, как на гитхабе?
c
portnov
эт конечно хорошо
эт конечно хорошо
t
в меркуриале как ж чуяли и сразу всем предлагали default её называть
Dv
Поскольку тесты очень долго идут (в хороший день 4 часа, в плохой — до суток), то у нас специальная автоматика, которая после тестов мержит бранч.
c
portnov
угу, ну суть в том, что до мержа
угу, ну суть в том, что до мержа
Dv
Ну и есть отдельные дейли тесты, которые уже в самом бранче ходят. За ними тоже следят.
c
portnov
а, во
а, во
Dv
А ещё есть бонус-тесты, которые сверхнадёжные, и поэтому должны десять раз подряд пройти, прежде чем что-то мержить.
Dv
Чтобы убедиться, что ты не смержил моргающие тесты в важные подсистемы.
Dv
Но моргающих тестов всё равно дофига, поэтому инфраструктура пытается прогонять основные тесты до четырёх раз, если они падают
c
portnov
а чья обязанность следить, что тесты проходят, и пинать кого надо?
а чья обязанность следить, что тесты проходят, и пинать кого надо?
c
portnov
тимлид? или спцеиальный человек?
тимлид? или спцеиальный человек?
Dv
Моя, наверное?
Dv
Вообще, я как-то не задумывался…