Size: a a a

2021 May 23

AM

Artem Molotov in PHP
Вполне.
источник

AM

Artem Molotov in PHP
Можешь ещё сюда посмотреть (надстройка) https://github.com/laminas/laminas-coding-standard
источник

М

Михаил in PHP
Хорошо
источник

М

Михаил in PHP
Может вспомнишь какую либо статью где рассказывается о правильной настройки phpcs и phpcbf на CI?
источник

М

Михаил in PHP
Найти что то не смог пока что статью такую
источник

AM

Artem Molotov in PHP
Создаёшь джобу в gitlab-ci.yml для всех веток кроме девелопа или мастера, к примеру, которая будет запускать composer cs-check. В таком случае мердж пул реквеста будет блокироваться. Если нужно разрешить мердж, то добавляешь к задаче allow_failure: true + добавляешь джобу для мастера/девелопа с composer cs-fix

Можно и сразу во всех ветках фиксить, но в таком случае разработчик будет вынужден пулить после каждого пуша (т.е. спровоцирует ряд конфликтов).

Ещё сюда можно глянуть https://github.com/micheh/phpcs-gitlab
источник

М

Михаил in PHP
Спасибо, ознакомлюсь )
источник

М

Михаил in PHP
Тупой вопрос мб
В голове пока не складывается.
Т.е. код стайл( php code_sniffer) будет применяться всегда только к файлам которые в коммите или мерже измененные, а не ко всем файлам в проекте?
источник

AM

Artem Molotov in PHP
Как настроишь. В вышеуказанном случае будет по всем, если я не туплю. Но ничто не мешает передавать список файлов, получив их через команды гита
источник

DT

Dmitriy Tkachenko in PHP
да, есть даже однострочник для применения фиксера к только измененным файлам
источник

DT

Dmitriy Tkachenko in PHP
его впринципе не сложно написать и самому
источник

DT

Dmitriy Tkachenko in PHP
а какой смысл? тут либо весь код по правилам кодстайла написан, либо это не важно
источник

М

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

М

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

М

Михаил in PHP
Я разбираюсь в этом, и пока не очень компетентен, поэтому посоветуйте, какой вариант лучше и надежнее
источник

М

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

DT

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

М

Михаил in PHP
Принял, тогда оставлю на весь проект
источник

OZ

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

М

Михаил in PHP
Принял, всем спасибо, помогли.
источник