Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
BoShurik opened this issue Jul 10, 2020 · 1 comment
Open
Assignees
Labels
improvement улучшение, идея

Comments

@BoShurik
Copy link

Напрямую эта зависимость не используется (все равно надо самому прописывать путь до правил), а для правильной работы библиотеки недостаточно просто иметь 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 несколько дольше идет обновление, т.к. минуется кеш.
Я бы предпочел добавить эти правила в ресурсы в этом случае, либо держать где-то отдельно глобально.

@BoShurik BoShurik changed the title [RFC] Перенести staticall/petrovich-php в suggest [RFC] Перенести cloudloyalty/petrovich-rules в suggest Jul 10, 2020
@staticall staticall self-assigned this Jul 15, 2020
@staticall staticall added the improvement улучшение, идея label Jul 15, 2020
@staticall
Copy link
Owner

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement улучшение, идея
Projects
None yet
Development

No branches or pull requests

2 participants