Комментарии 11
Не было проблем с переизбытком уведомлений?
Если на каждый чих делать уведомление то они через какое то время начнут просто игнорироваться
Идеально отправлять только уведомление падения джобы при мерж/pull реквесте с прожатой кнопкой "set auto merge "
То есть только тогда когда это необходимо
Уведомления на почту с Gitea при провале CIки не приходят, следовательно, переизбытка нет)
Ну, могу сказать, что уведомления в рабочий чат у нас не игнорируются, потому что в силу того, что коллектив у нас небольшой, то мы ждем коммитов друг друга и потом смотрим, что там появилось, чтобы понимать как с этим потом будем работать
Т.е. если есть пр (а мы узнаем об этом из уведомлений), то надо пойти посмотреть и апрувнуть, если все хорошо и все устраивает
Лично мне кажется, что у нас достаточно удобная система и в какой-то степени этот нотификатор - ее часть
От кстати, на счёт заголовка!
У вас же было гораздо более простое решение - сделать хук и выполнять Линт перед каждым push
Да и тесты туда можно сунуть, юниты по крайней мере
Не совсем понял проблему, у вас можно пушить напрямую в мастер минуя пул-реквесты? Если нет, то проблема упавшей сборки или тестов в пул-реквесте это проблема его владельца, замержить упавший пул-реквест нельзя, так что люди сами без всякой ругани приучаются к линтерам, если хотят чтоб их изменения в итоге попали в мастер.
Мне кажется, что вы не до конца поняли суть этого инструмента
Я вижу ее так:
сократить количество открытых и отслеживаемых ресурсов, для повышения концентрации;
перенести уведомления с почтового ящика в рабочий телеграм чат/тред;
добавить «наглядность» не только в вопросе линтеров, но и длительности пайплайнов и прочее для команды
Мое личное мнение заключается в том, что инструмент и правда очень удобен. Я перестал дергаться, чтобы проверить упал ли пайплайн или прошёл (не смотря на то, что использую пре-коммит, ведь бывают и другие причины, не правда ли?), а также выработал привычку: закомитил -> пушнул ->дождался результатов пайплайна -> открыл пр
перенести уведомления с почтового ящика
Ещё и при том, что уведомления об упавшем CI из GitHub может прийти с задержкой, а с нашего Gitea такие уведомления не идут вовсе.
Я суть инструмента прекрасно понимаю и на своем опыте знаю, что интеграция с рабочим мессенжером это классная тема, сильно удобнее, чем спамить в почтовый ящик. У меня на работе слак и я получаю уведомление, если в моем пул-реквесте упала сборка, или тесты, со ссылкой на билд, так что я сразу из мессенжера перехожу по ссылке и вижу что упало. Или когда мой пул-реквест заапруили и он готов к мержу в мастер. Или когда кто-то не только заапрувил, но и сразу замержил. Или когда мои уже замерженные в мастер изменения продеплоены на тестовой среде и готовы к использованию. Все это очень удобно.
Но я не понимаю сути изначальной проблемы с тем разрабом. Какая разница упоротый это тип, отказывающийся от линтеров, или нет, если его задача это замержить пул-реквест, а его нельзя замержить если проверка линтером упала? Зачем команде с ним ругаться? Я могу это понять только если в команде принято пушить в мастер напрямую, без пул-реквестов, то есть он периодически ломает мастер для всех, а не только лишь свою одну ветку.
Непонятно почему разработчик вообще что-то должен запускать руками, оно должно автоматически запускаться на создании пр и если не проходит, то мержа не будет.
И да, волновать состояние чужого пр и не должно
Команда игнорировала линтеры и я написал свой нотификатор