staticall/petrovich-php

[RFC] Перенести cloudloyalty/petrovich-rules в suggest

BoShurik opened this issue · 1 comments

BoShurik commented

Напрямую эта зависимость не используется (все равно надо самому прописывать путь до правил), а для правильной работы библиотеки недостаточно просто иметь staticall/petrovich-php в зависимостях.
В итоге для подключения пришлось использовать следующий composer.json

{
    "require": {
        "boshurik/twig-petrovich-extension": "^0.1.0",
        "cloudloyalty/petrovich-rules": "dev-master@dev",
    },
    "config": {
        "preferred-install": {
            "cloudloyalty/petrovich-rules": "source",
            "*": "dist"
        }
    },
    "minimum-stability": "stable",
}

Из-за source несколько дольше идет обновление, т.к. минуется кеш.
Я бы предпочел добавить эти правила в ресурсы в этом случае, либо держать где-то отдельно глобально.

staticall commented

Хорошая идея, спасибо!

Да, это должно решить проблему с обновлением правил. Но правила используются в тестах точно, через src/Petrovich/Loader.php::getVendorRulesFilePath()
Там на основе офф. правил проверяется склонения, чтобы интеграция ничего не сломала. По идее, надо тесты или переписать и необходимые куски правил вынести или хранить актуальную копию локально (что не хочется), или перенести в require-dev cloudloyalty/petrovich-rules и добавить в suggest

Надо будет подумать как лучше сделать