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

У нас проект собирается на Laravel. Стандарт PSR-12 Сейчас прогнал 1 файл дефолтный, и он там нашел несоответствие , т.е. он так все файлы поправить может

0

23.05.2021

#Миша Автор вопроса

Да и в плане производительности, это не слишком? После каждого пуша или мержа прогонять весь проект через код стайл и фиксер

0

23.05.2021

#Миша Автор вопроса

С этим я согласен, да. Но правильно ли что все сорцы может пофиксить под стандарт? Или это следствие использования стандарта ?

0

23.05.2021

Дмитрий Ткаченко

берешь последний коммит, запускаешь фиксер на весь проект, коммитишь и пушишь, и у тебя весь проект по кодстайлам

0

23.05.2021

Oleg Zenkovich

Это очень быстро прогоняется, не переживай)

0

23.05.2021

A 💬 · Accepted Answer

Создаёшь джобу в gitlab-ci.yml для всех веток кроме девелопа или мастера, к примеру, которая будет запускать composer cs-check. В таком случае мердж пул реквеста будет блокироваться. Если нужно разрешить мердж, то добавляешь к задаче allow_failure: true + добавляешь джобу для мастера/девелопа с composer cs-fix Можно и сразу во всех ветках фиксить, но в таком случае разработчик будет вынужден пулить после каждого пуша (т.е. спровоцирует ряд конфликтов). Ещё сюда можно глянуть https://github.com/micheh/phpcs-gitlab

169 похожих чатов

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

8 ответов

Похожие вопросы