EZ
права на мерж есть у всех. но только после того как пройдут все проверки
Ну даже если будешь прогонять тесты из мастера это тебя не спасет
Size: a a a
2020 November 11
EZ
Просто придется отдельным пром их выключить
RU
да, но это уже подозрительно
RU
и граничит с вредительством
RU
но в целом да, ты прав
EZ
и граничит с вредительством
Грань тонка
EZ
Это может быть другой пр, но с контентом
RU
дада, все так. но это уже многоходовочки - они на такое не способны
RU
к счастью
EZ
Ну хз, мне кажется это классический X-Y problem
RU
возможно
EZ
Опять таки, прогоняешь тесты из мастера -- новые тесты возможно не проходят
RU
у нас уже есть кастомное решение. yml файл с правами и проверками, который как раз читается из мастера. а чтобы изменить сам этот yml файл в нем же самом написано, что нужен аппрув "определенных людей из верушки". хотел бы перенести это как-то на экшны
RU
Опять таки, прогоняешь тесты из мастера -- новые тесты возможно не проходят
а я не про тесты. я в основном про ревьюверов
RU
то есть что-то типа github codeowners, но только enforced и более гибкое - не только по файлам, но и по разным условиям
RU
например бандл-сайз вырос больше чем на полпроцента в результате ПРа - назначить определенных людей на ревью и не пущать дальше, пока они не заапрувают
Dv
да, но сам воркфлоу файл берется из бранча или из мастера?
Из бранча.
RU
Из бранча.
я уже понял да
RU
Из бранча.
я вот кстати понять не могу, я щас запушил коммит с новым воркфлоу, и он у меня только с ручным триггером пока. но триггернуть я его не могу, потому что его не видно в workflows
RU
Из бранча.
и кстати, я понял, что ничто не мешает мне внутри самого экшна сходить в мастер бранч по апи и вытянуть все что мне надо