search

Chazovs / retailcrm-validator

MIT License
0 stars 1 forks source link

Review #4

Closed Neur0toxine closed 3 years ago

Neur0toxine commented 3 years ago

https://github.com/Chazovs/retailcrm-validator/blob/main/src/Validator/CrmUrlValidator.php#L147 - можно сократить так:

return in_array($domainHost, $this->getValidDomains($crmDomainsUrl), true);

Также падают phpcs, phpmd, phpstan и тесты. Для phpmd нет конфига (можно взять из API-клиента готовый). Также никак не настроены GitHub Actions для работы тестов и утилит проверки качества кода (в качестве шаблона можно взять конфиги пайплайнов из API-клиента).

Chazovs commented 3 years ago

исправил